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 ubiz...@gmail.com 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.

[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

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 hongjiu...@intel.com 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