Re: [PING][PATCH, AArch64] Add doloop_end pattern for -fmodulo-sched

2014-11-12 Thread Richard Henderson
On 11/12/2014 11:01 AM, Yangfei (Felix) wrote: > +(define_expand "doloop_end" >> + [(use (match_operand 0 "" "")) ; loop pseudo >> + (use (match_operand 1 "" ""))] ; label >> + "" >> + " >> +{ Drop the "" surrounding the { }. r~

Re: [PING][PATCH, AArch64] Add doloop_end pattern for -fmodulo-sched

2014-11-12 Thread Andrew Pinski
On Wed, Nov 12, 2014 at 2:54 AM, Jiong Wang wrote: > On 12/11/14 10:01, Yangfei (Felix) wrote: > >> + if (GET_MODE (operands[0]) != DImode) >> + FAIL; > > > shouldn't SImode be considered for -mabi=ilp32 ? I don't think it is needed for -mabi=ilp32 since word_mode is still DImode for -

Re: [PING][PATCH, AArch64] Add doloop_end pattern for -fmodulo-sched

2014-11-12 Thread Jiong Wang
On 12/11/14 10:01, Yangfei (Felix) wrote: + if (GET_MODE (operands[0]) != DImode) + FAIL; shouldn't SImode be considered for -mabi=ilp32 ?

[PING][PATCH, AArch64] Add doloop_end pattern for -fmodulo-sched

2014-11-12 Thread Yangfei (Felix)
Hello, Any comments on this patch? Thanks. > Hi, > This patch adds doloop_end standard pattern for AArch64 port so that > -fmodulo-sched can do its job. > Reg-tested for aarch64-linux-gnu with QEMU. OK for the trunk? > > > Index: gcc/ChangeLog > ===