Re: [PATCH] Fix for PR62037

2014-08-13 Thread Felix Yang
Hi Richard, Yeah, the testcase is there. Sorry for my mistake. Thank you for back-porting this patch to 4.8 & 4.9 branch. Cheers, Felix On Wed, Aug 13, 2014 at 5:30 PM, Richard Biener wrote: > On Tue, Aug 12, 2014 at 6:40 PM, Felix Yang wrote: >> Hi Richard, >> >> I find that you

Re: [PATCH] Fix for PR62037

2014-08-13 Thread Richard Biener
On Tue, Aug 12, 2014 at 6:40 PM, Felix Yang wrote: > Hi Richard, > > I find that you missed the testcase with when committing the patch. I don't think so. > GCC 4.8 & 4.9 branch also has the same issue, may be we need to > fix for them too? Yeah, I'll backport it. Thanks, Richard. > C

Re: [PATCH] Fix for PR62037

2014-08-12 Thread Felix Yang
Hi Richard, I find that you missed the testcase with when committing the patch. GCC 4.8 & 4.9 branch also has the same issue, may be we need to fix for them too? Cheers, Felix On Mon, Aug 11, 2014 at 7:24 PM, Richard Biener wrote: > On Sat, Aug 9, 2014 at 6:28 AM, Felix Yang wrote: >>

Re: [PATCH] Fix for PR62037

2014-08-11 Thread Richard Biener
On Sat, Aug 9, 2014 at 6:28 AM, Felix Yang wrote: > Attached please find the patch and testcase for PR62037. > > DEF1 can be a GIMPLE_NOP and gimple_bb will be NULL then. The patch > checks for that. > Bootstrapped on x86_64-suse-linux. OK for trunk? Please commit this > patch if it's OK. > Thank