Re: [PATCH] Fix another reg-stack recovery bug (PR target/84828)

2018-03-13 Thread Jakub Jelinek
On Tue, Mar 13, 2018 at 04:59:58AM -0700, H.J. Lu wrote: > >> The testcase still ICEs on i686-linux (preexisting bug, ICEs the same > >> without this patch or even before the previous patch), will handle that > >> tomorrow. > I got: See the above? Or do you get this with -m64 too, rather than

Re: [PATCH] Fix another reg-stack recovery bug (PR target/84828)

2018-03-13 Thread H.J. Lu
On Tue, Mar 13, 2018 at 5:02 AM, Jakub Jelinek wrote: > On Tue, Mar 13, 2018 at 04:59:58AM -0700, H.J. Lu wrote: >> >> The testcase still ICEs on i686-linux (preexisting bug, ICEs the same >> >> without this patch or even before the previous patch), will handle that >> >>

Re: [PATCH] Fix another reg-stack recovery bug (PR target/84828)

2018-03-13 Thread H.J. Lu
On Tue, Mar 13, 2018 at 12:55 AM, Uros Bizjak wrote: > On Mon, Mar 12, 2018 at 10:10 PM, Jakub Jelinek wrote: >> Hi! >> >> As Martin reported, the same testcase added recently ICEs differently >> with different options on x86_64-linux, the problem is that we

Re: [PATCH] Fix another reg-stack recovery bug (PR target/84828)

2018-03-13 Thread Uros Bizjak
On Mon, Mar 12, 2018 at 10:10 PM, Jakub Jelinek wrote: > Hi! > > As Martin reported, the same testcase added recently ICEs differently > with different options on x86_64-linux, the problem is that we > sometimes emit insns before a CODE_LABEL of the next bb rather than after >