Re: [PATCH] Fix up RTL DCE find_call_stack_args (PR rtl-optimization/89965, take 2)

2019-04-12 Thread Jeff Law
On 4/11/19 3:04 AM, Jakub Jelinek wrote: > On Thu, Apr 11, 2019 at 09:54:24AM +0200, Richard Biener wrote: >>> Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? >>> >>> During those 2 bootstraps/regtests, data.load_found has been set just >>> on the new testcase on ia32. >> >> Hm

Re: [PATCH] Fix up RTL DCE find_call_stack_args (PR rtl-optimization/89965, take 2)

2019-04-12 Thread Jeff Law
On 4/11/19 3:26 AM, Richard Biener wrote: > On Thu, 11 Apr 2019, Jakub Jelinek wrote: > >> On Thu, Apr 11, 2019 at 09:54:24AM +0200, Richard Biener wrote: Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? During those 2 bootstraps/regtests, data.load_found has bee

Re: [PATCH] Fix up RTL DCE find_call_stack_args (PR rtl-optimization/89965, take 2)

2019-04-11 Thread Richard Biener
On Thu, 11 Apr 2019, Jakub Jelinek wrote: > On Thu, Apr 11, 2019 at 09:54:24AM +0200, Richard Biener wrote: > > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? > > > > > > During those 2 bootstraps/regtests, data.load_found has been set just > > > on the new testcase on ia3

[PATCH] Fix up RTL DCE find_call_stack_args (PR rtl-optimization/89965, take 2)

2019-04-11 Thread Jakub Jelinek
On Thu, Apr 11, 2019 at 09:54:24AM +0200, Richard Biener wrote: > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? > > > > During those 2 bootstraps/regtests, data.load_found has been set just > > on the new testcase on ia32. > > Hmm, I wonder whether we really need to DCE c