* [PATCH 0/3] aarch64: ls64 builtin fixes [PR110100,PR110132]
@ 2023-06-07 13:59 Alex Coplan
2023-06-07 15:51 ` Richard Sandiford
0 siblings, 1 reply; 2+ messages in thread
From: Alex Coplan @ 2023-06-07 13:59 UTC (permalink / raw)
To: gcc-patches; +Cc: Richard Earnshaw, Richard Sandiford, Kyrylo Tkachov
Hi,
This patch series fixes various defects with the FEAT_LS64 ACLE
implementation in the AArch64 backend.
The series is organised as follows:
- Patch 1/3 fixes whitespace errors in the existing code.
- Patch 2/3 fixes PR110100 where we generate wrong code for the st64b
builtin.
- Patch 3/3 fixes PR110132, allowing the compiler to define the ACLE builtins
directly, and also makes the builtins work under LTO.
Bootstrapped/regtested as a series on aarch64-linux-gnu. OK for trunk
and backports back to GCC 12?
Thanks,
Alex
Alex Coplan (3):
aarch64: Fix whitespace in ls64 builtin implementation [PR110100]
aarch64: Fix wrong code with st64b builtin [PR110100]
aarch64: Allow compiler to define ls64 builtins [PR110132]
gcc/config/aarch64/aarch64-builtins.cc | 88 +++++++++++--------
gcc/config/aarch64/aarch64.md | 24 ++---
gcc/config/aarch64/arm_acle.h | 33 -------
.../g++.target/aarch64/acle/acle.exp | 35 ++++++++
gcc/testsuite/g++.target/aarch64/acle/ls64.C | 10 +++
.../g++.target/aarch64/acle/ls64_lto.C | 10 +++
.../gcc.target/aarch64/acle/ls64_lto.c | 10 +++
.../gcc.target/aarch64/acle/pr110100.c | 7 ++
.../gcc.target/aarch64/acle/pr110132.c | 15 ++++
gcc/testsuite/lib/target-supports.exp | 2 +-
10 files changed, 151 insertions(+), 83 deletions(-)
create mode 100644 gcc/testsuite/g++.target/aarch64/acle/acle.exp
create mode 100644 gcc/testsuite/g++.target/aarch64/acle/ls64.C
create mode 100644 gcc/testsuite/g++.target/aarch64/acle/ls64_lto.C
create mode 100644 gcc/testsuite/gcc.target/aarch64/acle/ls64_lto.c
create mode 100644 gcc/testsuite/gcc.target/aarch64/acle/pr110100.c
create mode 100644 gcc/testsuite/gcc.target/aarch64/acle/pr110132.c
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH 0/3] aarch64: ls64 builtin fixes [PR110100,PR110132]
2023-06-07 13:59 [PATCH 0/3] aarch64: ls64 builtin fixes [PR110100,PR110132] Alex Coplan
@ 2023-06-07 15:51 ` Richard Sandiford
0 siblings, 0 replies; 2+ messages in thread
From: Richard Sandiford @ 2023-06-07 15:51 UTC (permalink / raw)
To: Alex Coplan; +Cc: gcc-patches, Richard Earnshaw, Kyrylo Tkachov
Alex Coplan <alex.coplan@arm.com> writes:
> Hi,
>
> This patch series fixes various defects with the FEAT_LS64 ACLE
> implementation in the AArch64 backend.
>
> The series is organised as follows:
>
> - Patch 1/3 fixes whitespace errors in the existing code.
> - Patch 2/3 fixes PR110100 where we generate wrong code for the st64b
> builtin.
> - Patch 3/3 fixes PR110132, allowing the compiler to define the ACLE builtins
> directly, and also makes the builtins work under LTO.
>
> Bootstrapped/regtested as a series on aarch64-linux-gnu. OK for trunk
> and backports back to GCC 12?
Yeah, OK for trunk and branches. Thanks for fixing this.
Richard
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-06-07 15:51 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-06-07 13:59 [PATCH 0/3] aarch64: ls64 builtin fixes [PR110100,PR110132] Alex Coplan
2023-06-07 15:51 ` Richard Sandiford
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).