Re: [PATCH] PR target/66813: gcc.target/i386/asm-flag-5.c failed with -march=pentium

2015-07-10 Thread Uros Bizjak
On Thu, Jul 9, 2015 at 3:15 PM, Uros Bizjak wrote: >> gen_rtx_ZERO_EXTEND isn't suitable in ix86_md_asm_adjust since ZERO_EXTEND >> may be expaned. We should call gen_zero_extendqiXi2 instead. >> >> OK for trunk? > > No, your patch will clobber flags when multiple flag outputs are used. > > (I p

Re: [PATCH] PR target/66813: gcc.target/i386/asm-flag-5.c failed with -march=pentium

2015-07-09 Thread Uros Bizjak
On Thu, Jul 9, 2015 at 3:10 PM, H.J. Lu wrote: > gen_rtx_ZERO_EXTEND isn't suitable in ix86_md_asm_adjust since ZERO_EXTEND > may be expaned. We should call gen_zero_extendqiXi2 instead. > > OK for trunk? No, your patch will clobber flags when multiple flag outputs are used. (I plan to rewrite

[PATCH] PR target/66813: gcc.target/i386/asm-flag-5.c failed with -march=pentium

2015-07-09 Thread H.J. Lu
gen_rtx_ZERO_EXTEND isn't suitable in ix86_md_asm_adjust since ZERO_EXTEND may be expaned. We should call gen_zero_extendqiXi2 instead. OK for trunk? H.J. --- gcc/ PR target/66813 * config/i386/i386.c (ix86_md_asm_adjust): Replace gen_rtx_ZERO_EXTEND with gen_zero_extend