Re: [PATCH] vect: more oversized bitmask fixups

2024-03-27 Thread Thomas Schwinge
Hi! On 2024-03-22T14:15:36+, Andrew Stubbs wrote: > On 22/03/2024 08:43, Richard Biener wrote: > Thanks, here's what I pushed. > vect: more oversized bitmask fixups > > These patches fix up a failure in testcase vect/tsvc/vect-tsvc-s278.c when > configured to use V32 instead of V64 (I plan t

Re: [PATCH] vect: more oversized bitmask fixups

2024-03-22 Thread Andrew Stubbs
On 22/03/2024 08:43, Richard Biener wrote: I'll note that we don't pass 'val' there and 'val' is unfortunately not documented - what's it supposed to be? I think I placed the original fix in do_compare_and_jump because we have the full into available there. So what's the do_compare_rtx_and_j

Re: [PATCH] vect: more oversized bitmask fixups

2024-03-22 Thread Richard Biener
On Thu, Mar 21, 2024 at 5:07 PM Andrew Stubbs wrote: > > On 21/03/2024 15:18, Richard Biener wrote: > > On Thu, Mar 21, 2024 at 3:23 PM Andrew Stubbs wrote: > >> > >> My previous patch to fix this problem with xor was rejected because we > >> want to fix these issues only at the point of use. Th

Re: [PATCH] vect: more oversized bitmask fixups

2024-03-21 Thread Andrew Stubbs
On 21/03/2024 15:18, Richard Biener wrote: On Thu, Mar 21, 2024 at 3:23 PM Andrew Stubbs wrote: My previous patch to fix this problem with xor was rejected because we want to fix these issues only at the point of use. That patch produced slightly better code, in this example, but this works t

Re: [PATCH] vect: more oversized bitmask fixups

2024-03-21 Thread Richard Biener
On Thu, Mar 21, 2024 at 3:23 PM Andrew Stubbs wrote: > > My previous patch to fix this problem with xor was rejected because we > want to fix these issues only at the point of use. That patch produced > slightly better code, in this example, but this works too > > These patches fix up a failu

[PATCH] vect: more oversized bitmask fixups

2024-03-21 Thread Andrew Stubbs
My previous patch to fix this problem with xor was rejected because we want to fix these issues only at the point of use. That patch produced slightly better code, in this example, but this works too These patches fix up a failure in testcase vect/tsvc/vect-tsvc-s278.c when configured to use