Re: Handle GIMPLE NOPs in is_maybe_undefined (PR, tree-optimization/79529).

2017-02-17 Thread Richard Biener
On Fri, Feb 17, 2017 at 10:55 AM, Martin Liška wrote: > On 02/16/2017 12:34 PM, Richard Biener wrote: >> Yes, we should handle all of the "hidden initialized" cases at >> >> /* A PARM_DECL will not have an SSA_NAME_DEF_STMT. Parameters >> get their initial value from function entry

Re: Handle GIMPLE NOPs in is_maybe_undefined (PR, tree-optimization/79529).

2017-02-17 Thread Martin Liška
On 02/16/2017 12:34 PM, Richard Biener wrote: > Yes, we should handle all of the "hidden initialized" cases at > > /* A PARM_DECL will not have an SSA_NAME_DEF_STMT. Parameters > get their initial value from function entry. */ > if (SSA_NAME_VAR (t) && TREE_CODE (SSA_NAME_VA

Re: Handle GIMPLE NOPs in is_maybe_undefined (PR, tree-optimization/79529).

2017-02-16 Thread Richard Biener
On Thu, Feb 16, 2017 at 11:45 AM, Aldy Hernandez wrote: > On 02/16/2017 03:46 AM, Martin Liška wrote: >> >> On 02/15/2017 05:06 PM, Aldy Hernandez wrote: >>> >>> On 02/15/2017 09:49 AM, Martin Liška wrote: Hi. As mentioned in the PR, gimple nops are wrongly handled in is_m

Re: Handle GIMPLE NOPs in is_maybe_undefined (PR, tree-optimization/79529).

2017-02-16 Thread Aldy Hernandez
On 02/16/2017 03:46 AM, Martin Liška wrote: On 02/15/2017 05:06 PM, Aldy Hernandez wrote: On 02/15/2017 09:49 AM, Martin Liška wrote: Hi. As mentioned in the PR, gimple nops are wrongly handled in is_maybe_undefined function. Patch can bootstrap on ppc64le-redhat-linux and survives regression

Re: Handle GIMPLE NOPs in is_maybe_undefined (PR, tree-optimization/79529).

2017-02-16 Thread Martin Liška
On 02/15/2017 05:06 PM, Aldy Hernandez wrote: > On 02/15/2017 09:49 AM, Martin Liška wrote: >> Hi. >> >> As mentioned in the PR, gimple nops are wrongly handled in >> is_maybe_undefined function. >> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. > >> >> gimple *d

Re: Handle GIMPLE NOPs in is_maybe_undefined (PR, tree-optimization/79529).

2017-02-15 Thread Aldy Hernandez
On 02/15/2017 09:49 AM, Martin Liška wrote: Hi. As mentioned in the PR, gimple nops are wrongly handled in is_maybe_undefined function. Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. gimple *def = SSA_NAME_DEF_STMT (t); + if (!def || gimple_nop_p (def

Handle GIMPLE NOPs in is_maybe_undefined (PR, tree-optimization/79529).

2017-02-15 Thread Martin Liška
Wed, 15 Feb 2017 13:46:38 +0100 Subject: [PATCH] Handle GIMPLE NOPs in is_maybe_undefined (PR tree-optimization/79529). gcc/ChangeLog: 2017-02-15 Martin Liska PR tree-optimization/79529 * tree-ssa-loop-unswitch.c (is_maybe_undefined): Bail out when spotting a gimple NOP. --- gcc/tree-ssa-l