Re: [PATCH v2 1/2] Enhance cse_insn to handle all-zeros and all-ones for vector mode.

On Tue, Aug 27, 2024 at 5:20 AM liuhongt wrote: > > > You are possibly overwriting src_related_elt - I'd suggest to either break > > here or do the loop below for each found elt? > Changed. > > > Do we know that will always succeed? > 1) validate_subreg allows subreg for 2 vector modes with same c

[PATCH v2 1/2] Enhance cse_insn to handle all-zeros and all-ones for vector mode.

> You are possibly overwriting src_related_elt - I'd suggest to either break > here or do the loop below for each found elt? Changed. > Do we know that will always succeed? 1) validate_subreg allows subreg for 2 vector modes with same component modes. 2) gen_lowpart in cse.cc is defined as gen_low