Re: [PATCH] Fix enum operands exchange in ipa-inline.c

2015-01-21 Thread Rainer Orth
Richard Biener writes: > On Mon, Jan 12, 2015 at 10:36 AM, Hurugalawadi, Naveen > wrote: >> Hi Richard, >> >> Thanks for the quick review and comments. >> >> Please find attached the modified patch as per your suggestion. > > Ok. Unfortunately, this patch causes an ICE on Solaris 10: FAIL: g++

Re: [PATCH] Fix enum operands exchange in ipa-inline.c

2015-01-12 Thread Richard Biener
_ > From: Richard Biener > Sent: Monday, January 12, 2015 2:48 PM > To: Hurugalawadi, Naveen > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [PATCH] Fix enum operands exchange in ipa-inline.c > > On Mon, Jan 12, 2015 at 7:58 AM, Hurugalawadi, Naveen > wrote: >> Hi, >> >

Re: [PATCH] Fix enum operands exchange in ipa-inline.c

2015-01-12 Thread Hurugalawadi, Naveen
: Re: [PATCH] Fix enum operands exchange in ipa-inline.c On Mon, Jan 12, 2015 at 7:58 AM, Hurugalawadi, Naveen wrote: > Hi, > > Sorry, Had forgot the ChangeLog. Ok, but please properly wrap the long lines, put '? gimple_...' on a new one. Thanks, Richard. > ChangeLog &g

Re: [PATCH] Fix enum operands exchange in ipa-inline.c

2015-01-12 Thread Richard Biener
On Mon, Jan 12, 2015 at 7:58 AM, Hurugalawadi, Naveen wrote: > Hi, > > Sorry, Had forgot the ChangeLog. Ok, but please properly wrap the long lines, put '? gimple_...' on a new one. Thanks, Richard. > ChangeLog > 2015-01-12 Naveen H.S > > * ipa-inline.c (inline_small_functions): Swap

Re: [PATCH] Fix enum operands exchange in ipa-inline.c

2015-01-11 Thread Hurugalawadi, Naveen
Hi, Sorry, Had forgot the ChangeLog. ChangeLog 2015-01-12 Naveen H.S * ipa-inline.c (inline_small_functions): Swap the operands in enum. Thanks, Naveen

[PATCH] Fix enum operands exchange in ipa-inline.c

2015-01-11 Thread Hurugalawadi, Naveen
Hi, Please find attached the patch that fixes swap of operands to enum in ipa-inline.c. This issue popped up when running dhrystone with -fdump-ipa-all. Please review the patch and let me know it its okay? Regression tested on aarch64-elf. Thanks, Naveen--- a/gcc/ipa-inline.c 2015-01-12 11:04