Re: [PATCH v8] PR middle-end/60281

2014-04-18 Thread Jakub Jelinek
On Fri, Apr 18, 2014 at 06:05:35PM +0800, lin zuojian wrote: > Hi Jakub, > > On Fri, Apr 18, 2014 at 09:09:17AM +0200, Jakub Jelinek wrote: > > Extra line missing before the PR line. > Should I post PATCH v9 or someone helps adding one when committing > the patch? No need for that, that was meant

Re: [PATCH v8] PR middle-end/60281

2014-04-18 Thread lin zuojian
Hi Jakub, On Fri, Apr 18, 2014 at 09:09:17AM +0200, Jakub Jelinek wrote: > Extra line missing before the PR line. Should I post PATCH v9 or someone helps adding one when committing the patch? -- Regards lin zuojian

Re: [PATCH v8] PR middle-end/60281

2014-04-18 Thread Jakub Jelinek
On Fri, Apr 18, 2014 at 12:21:50PM +0800, lin zuojian wrote: > Here is the patch after the Jakub's review, and Jakub helps with the > coding style. > --- a/gcc/ChangeLog > +++ b/gcc/ChangeLog > @@ -1,3 +1,12 @@ > +2014-04-18 Lin Zuojian > + PR middle-end/60281 Extra line missing

RE: [PATCH v8] PR middle-end/60281

2014-04-17 Thread Bernd Edlinger
Hi Jakub, I can take that task over and will boot-strap with all languages and run the test suite on my armv7-linux-gnueabihf system. But that will take until next week as it is currently occupied with other tests. Can you please review Lin's latest patch and give your OK for check-in on trunk a

Re: [PATCH v8] PR middle-end/60281

2014-04-17 Thread lin zuojian
Hi Bernd, a) On which target(s) did you boot-strap your patch? I just run it on x86, can't run it on ARM, because Android is not a posix system, nor a System V compatible system. And my code does not effect x86. b) Did you run the testsuite? Yes, but again my code does not effect x