Re: [PATCH] store-merging: Handle vector CONSTRUCTORs using bswap [PR96239]

2021-01-05 Thread Jakub Jelinek via Gcc-patches
On Tue, Jan 05, 2021 at 01:55:21PM +0100, Richard Biener wrote: > > Note, I have no idea why the bswap code needs TODO_update_ssa if it changed > > things, for the vuses it copies them from the surrounding vuses, which looks > > correct to me. Perhaps because it uses force_gimple_operand_gsi* in a

Re: [PATCH] store-merging: Handle vector CONSTRUCTORs using bswap [PR96239]

2021-01-05 Thread Richard Biener
On Thu, 17 Dec 2020, Jakub Jelinek wrote: > On Wed, Dec 16, 2020 at 09:29:31AM +0100, Richard Biener wrote: > > I think it probably makes sense to have some helper split out that > > collects & classifies vector constructor components we can use from > > both forwprop (where matching the V_C_E fro

[PATCH] store-merging: Handle vector CONSTRUCTORs using bswap [PR96239]

2020-12-17 Thread Jakub Jelinek via Gcc-patches
On Wed, Dec 16, 2020 at 09:29:31AM +0100, Richard Biener wrote: > I think it probably makes sense to have some helper split out that > collects & classifies vector constructor components we can use from > both forwprop (where matching the V_C_E from integer could be done > as well IMHO) and bswap (