diff options
| author | Tamar Christina <tamar.christina@arm.com> | 2025-03-04 11:15:26 +0000 |
|---|---|---|
| committer | Tamar Christina <tamar.christina@arm.com> | 2025-03-04 11:15:52 +0000 |
| commit | d883f3233c4b7e0dce52539a12dfcccc8aff43e4 (patch) | |
| tree | 2f28ac04844632f1b7f997fa443216cd4997deb7 /gcc | |
| parent | ac16d6d74fcb4ca10c939b00782b4dfada666273 (diff) | |
| download | gcc-d883f3233c4b7e0dce52539a12dfcccc8aff43e4.zip gcc-d883f3233c4b7e0dce52539a12dfcccc8aff43e4.tar.gz gcc-d883f3233c4b7e0dce52539a12dfcccc8aff43e4.tar.bz2 | |
aarch64: force operand to fresh register to avoid subreg issues [PR118892]
When the input is already a subreg and we try to make a paradoxical
subreg out of it for copysign this can fail if it violates the subreg
relationship.
Use force_lowpart_subreg instead of lowpart_subreg to then force the
results to a register instead of ICEing.
gcc/ChangeLog:
PR target/118892
* config/aarch64/aarch64.md (copysign<GPF:mode>3): Use
force_lowpart_subreg instead of lowpart_subreg.
gcc/testsuite/ChangeLog:
PR target/118892
* gcc.target/aarch64/copysign-pr118892.c: New test.
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/config/aarch64/aarch64.md | 2 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.target/aarch64/copysign-pr118892.c | 11 |
2 files changed, 12 insertions, 1 deletions
diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md index cfe730f..b10059e 100644 --- a/gcc/config/aarch64/aarch64.md +++ b/gcc/config/aarch64/aarch64.md @@ -7480,7 +7480,7 @@ { emit_insn (gen_ior<vq_int_equiv>3 ( lowpart_subreg (<VQ_INT_EQUIV>mode, operands[0], <MODE>mode), - lowpart_subreg (<VQ_INT_EQUIV>mode, operands[1], <MODE>mode), + force_lowpart_subreg (<VQ_INT_EQUIV>mode, operands[1], <MODE>mode), v_bitmask)); DONE; } diff --git a/gcc/testsuite/gcc.target/aarch64/copysign-pr118892.c b/gcc/testsuite/gcc.target/aarch64/copysign-pr118892.c new file mode 100644 index 0000000..adfa30d --- /dev/null +++ b/gcc/testsuite/gcc.target/aarch64/copysign-pr118892.c @@ -0,0 +1,11 @@ +/* { dg-do compile } */ +/* { dg-options "-Ofast" } */ + +double l(); +double f() +{ + double t6[2] = {l(), l()}; + double t7[2]; + __builtin_memcpy(&t7, &t6, sizeof(t6)); + return -__builtin_fabs(t7[1]); +} |