[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Tinycc-devel] [PATCH] accepting a 'b' suffix at the end of all the
Re: [Tinycc-devel] [PATCH] accepting a 'b' suffix at the end of all the setcc instructions
Sun, 04 Jan 2015 01:18:33 +0000
KMail/4.14.1 (Linux/2.6.38-ac2-ac100; KDE/4.14.2; armv7l; ; )
Le samedi 3 janvier 2015, 09:34:14 Sergey Korshunoff a écrit :
> Allow tcc to understand a setob,... opcodes as alias to seto,...
> .html This is fix PR8686 for llvm: accepting a 'b' suffix at the end of all
> the setcc instructions
As said previously I'm not very familiar with this part of the code but your
change looks good. I just have one suggestion:
rename DEF_ASMTEST by DEF_ASMTEST_SUF and make it take a third parameter that
is concatenated at the end. Then make DEF_ASMTEST call DEF_ASMTEST_SUF and you
can define DEF_ASMTEST_SET with DEF_ASM_SUF as well using less lines overall.
By the way, is it normal that there is only 2 variants of set for x86_64 but
so many for i386?