Re: [PATCH] Remove dead vector comparisons

2020-07-27 Thread Richard Biener via Gcc-patches
On Mon, Jul 27, 2020 at 2:50 PM Martin Liška wrote: > > On 7/27/20 1:22 PM, Richard Biener wrote: > > I wonder what happens if we make vector lowering not allow the compare > > expanded via expand_vec_cond_expr_p? > > So the following patch survives bootstrap and tests on x86_64-linux-gnu: > > dif

Re: [PATCH] Remove dead vector comparisons

2020-07-27 Thread Martin Liška
On 7/27/20 1:22 PM, Richard Biener wrote: I wonder what happens if we make vector lowering not allow the compare expanded via expand_vec_cond_expr_p? So the following patch survives bootstrap and tests on x86_64-linux-gnu: diff --git a/gcc/tree-vect-generic.c b/gcc/tree-vect-generic.c index f8

Re: [PATCH] Remove dead vector comparisons

2020-07-27 Thread Richard Biener via Gcc-patches
On Mon, Jul 27, 2020 at 11:32 AM Martin Liška wrote: > > On 7/10/20 10:24 AM, Richard Biener wrote: > > On Fri, Jul 10, 2020 at 9:50 AM Martin Liška wrote: > >> > >> As mentioned in the PR, we need to clean up orphan vector comparisons > >> that tend to happen be gimplification of VEC_COND_EXPR.

Re: [PATCH] Remove dead vector comparisons

2020-07-27 Thread Martin Liška
On 7/10/20 10:24 AM, Richard Biener wrote: On Fri, Jul 10, 2020 at 9:50 AM Martin Liška wrote: As mentioned in the PR, we need to clean up orphan vector comparisons that tend to happen be gimplification of VEC_COND_EXPR. I've done that easily in expand_vector_comparison where I add these to a

Re: [PATCH] Remove dead vector comparisons

2020-07-10 Thread Richard Biener via Gcc-patches
On Fri, Jul 10, 2020 at 9:50 AM Martin Liška wrote: > > As mentioned in the PR, we need to clean up orphan vector comparisons > that tend to happen be gimplification of VEC_COND_EXPR. > > I've done that easily in expand_vector_comparison where I add these > to a bitmap used in simple DCE. > > Patc

[PATCH] Remove dead vector comparisons

2020-07-10 Thread Martin Liška
As mentioned in the PR, we need to clean up orphan vector comparisons that tend to happen be gimplification of VEC_COND_EXPR. I've done that easily in expand_vector_comparison where I add these to a bitmap used in simple DCE. Patch can bootstrap on x86_64-linux-gnu and survives regression tests.