public inbox for gcc-cvs@sourceware.org
help / color / mirror / Atom feed
* [gcc(refs/vendors/ARM/heads/morello)] aarch64: Fix SVE ACLE regressions from label/symbol_ref patch
@ 2022-05-06 14:45 Matthew Malcomson
0 siblings, 0 replies; only message in thread
From: Matthew Malcomson @ 2022-05-06 14:45 UTC (permalink / raw)
To: gcc-cvs
https://gcc.gnu.org/g:ddcc575cad5cd730f3629c767f10bb358f46f90c
commit ddcc575cad5cd730f3629c767f10bb358f46f90c
Author: Richard Sandiford <richard.sandiford@arm.com>
Date: Thu Apr 28 17:05:52 2022 +0100
aarch64: Fix SVE ACLE regressions from label/symbol_ref patch
7e31e09b7dcb31114114b5719c9f198c013da0e4 regressed several
SVE ACLE tests because the vec_flags in the newly split-out
aarch64_valid_ldr_str_offset_p failed to copy the clearing
of the VEC_PARTIAL flag, which defeated the two later == tests.
Diff:
---
gcc/config/aarch64/aarch64.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c
index d953278d920..852564b8f0c 100644
--- a/gcc/config/aarch64/aarch64.c
+++ b/gcc/config/aarch64/aarch64.c
@@ -9640,7 +9640,9 @@ aarch64_valid_ldr_str_offset_p (machine_mode mode, bool alt_base_p,
poly_int64 offset,
aarch64_addr_query_type type)
{
+ /* See the comment in aarch64_classify_address. */
unsigned int vec_flags = aarch64_classify_vector_mode (mode);
+ vec_flags &= ~VEC_PARTIAL;
/* Make "m" use the LD1 offset range for SVE data modes, so
that pre-RTL optimizers like ivopts will work to that
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2022-05-06 14:45 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-06 14:45 [gcc(refs/vendors/ARM/heads/morello)] aarch64: Fix SVE ACLE regressions from label/symbol_ref patch Matthew Malcomson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).