[Bug target/110100] __builtin_aarch64_st64b stores to the wrong address
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110100 Alex Coplan changed: What|Removed |Added Resolution|--- |FIXED Status|ASSIGNED|RESOLVED --- Comment #10 from Alex Coplan --- Should be fixed on all affected branches, thanks for the report.
[Bug target/110100] __builtin_aarch64_st64b stores to the wrong address
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110100 --- Comment #9 from CVS Commits --- The releases/gcc-12 branch has been updated by Alex Coplan : https://gcc.gnu.org/g:0112ed013847ca9dbef4ba21f1c3f94c5bbe310b commit r12-9719-g0112ed013847ca9dbef4ba21f1c3f94c5bbe310b Author: Alex Coplan Date: Tue Jun 6 11:52:19 2023 +0100 aarch64: Fix wrong code with st64b builtin [PR110100] The st64b pattern incorrectly had an output constraint on the register operand containing the destination address for the store, leading to wrong code. This patch fixes that. gcc/ChangeLog: PR target/110100 * config/aarch64/aarch64-builtins.cc (aarch64_expand_builtin_ls64): Use input operand for the destination address. * config/aarch64/aarch64.md (st64b): Fix constraint on address operand. gcc/testsuite/ChangeLog: PR target/110100 * gcc.target/aarch64/acle/pr110100.c: New test. (cherry picked from commit 737a0b749a7bc3e7cb904ea2d4b18dc130514b85)
[Bug target/110100] __builtin_aarch64_st64b stores to the wrong address
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110100 --- Comment #8 from CVS Commits --- The releases/gcc-12 branch has been updated by Alex Coplan : https://gcc.gnu.org/g:c0ab0d4af51382d9d8d4e6b026865842d8e06d7e commit r12-9718-gc0ab0d4af51382d9d8d4e6b026865842d8e06d7e Author: Alex Coplan Date: Tue Jun 6 11:04:45 2023 +0100 aarch64: Fix whitespace in ls64 builtin implementation [PR110100] The ls64 builtin code was using incorrect GNU style with eight spaces where there should be a tab. Fixed thusly. gcc/ChangeLog: PR target/110100 * config/aarch64/aarch64-builtins.cc (aarch64_init_ls64_builtins_types): Replace eight consecutive spaces with tabs. (aarch64_init_ls64_builtins): Likewise. (aarch64_expand_builtin_ls64): Likewise. * config/aarch64/aarch64.md (ld64b): Likewise. (st64b): Likewise. (st64bv): Likewise (st64bv0): Likewise. (cherry picked from commit 713613541254039a34e1dd8fd4a613a299af1fd6)
[Bug target/110100] __builtin_aarch64_st64b stores to the wrong address
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110100 --- Comment #7 from CVS Commits --- The releases/gcc-13 branch has been updated by Alex Coplan : https://gcc.gnu.org/g:9df688cbf908adc43e92bd012dafa88680ea11dc commit r13-7461-g9df688cbf908adc43e92bd012dafa88680ea11dc Author: Alex Coplan Date: Tue Jun 6 11:52:19 2023 +0100 aarch64: Fix wrong code with st64b builtin [PR110100] The st64b pattern incorrectly had an output constraint on the register operand containing the destination address for the store, leading to wrong code. This patch fixes that. gcc/ChangeLog: PR target/110100 * config/aarch64/aarch64-builtins.cc (aarch64_expand_builtin_ls64): Use input operand for the destination address. * config/aarch64/aarch64.md (st64b): Fix constraint on address operand. gcc/testsuite/ChangeLog: PR target/110100 * gcc.target/aarch64/acle/pr110100.c: New test. (cherry picked from commit 737a0b749a7bc3e7cb904ea2d4b18dc130514b85)
[Bug target/110100] __builtin_aarch64_st64b stores to the wrong address
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110100 --- Comment #6 from CVS Commits --- The releases/gcc-13 branch has been updated by Alex Coplan : https://gcc.gnu.org/g:ff00fa1914e42d6b9c45cb36a5c99f94c4133cba commit r13-7460-gff00fa1914e42d6b9c45cb36a5c99f94c4133cba Author: Alex Coplan Date: Tue Jun 6 11:04:45 2023 +0100 aarch64: Fix whitespace in ls64 builtin implementation [PR110100] The ls64 builtin code was using incorrect GNU style with eight spaces where there should be a tab. Fixed thusly. gcc/ChangeLog: PR target/110100 * config/aarch64/aarch64-builtins.cc (aarch64_init_ls64_builtins_types): Replace eight consecutive spaces with tabs. (aarch64_init_ls64_builtins): Likewise. (aarch64_expand_builtin_ls64): Likewise. * config/aarch64/aarch64.md (ld64b): Likewise. (st64b): Likewise. (st64bv): Likewise (st64bv0): Likewise. (cherry picked from commit 713613541254039a34e1dd8fd4a613a299af1fd6)
[Bug target/110100] __builtin_aarch64_st64b stores to the wrong address
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110100 --- Comment #5 from Alex Coplan --- Fixed on trunk for GCC 14, keeping open for backports (I think we need this back to GCC 12).
[Bug target/110100] __builtin_aarch64_st64b stores to the wrong address
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110100 --- Comment #4 from CVS Commits --- The master branch has been updated by Alex Coplan : https://gcc.gnu.org/g:737a0b749a7bc3e7cb904ea2d4b18dc130514b85 commit r14-1616-g737a0b749a7bc3e7cb904ea2d4b18dc130514b85 Author: Alex Coplan Date: Tue Jun 6 11:52:19 2023 +0100 aarch64: Fix wrong code with st64b builtin [PR110100] The st64b pattern incorrectly had an output constraint on the register operand containing the destination address for the store, leading to wrong code. This patch fixes that. gcc/ChangeLog: PR target/110100 * config/aarch64/aarch64-builtins.cc (aarch64_expand_builtin_ls64): Use input operand for the destination address. * config/aarch64/aarch64.md (st64b): Fix constraint on address operand. gcc/testsuite/ChangeLog: PR target/110100 * gcc.target/aarch64/acle/pr110100.c: New test.
[Bug target/110100] __builtin_aarch64_st64b stores to the wrong address
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110100 --- Comment #3 from CVS Commits --- The master branch has been updated by Alex Coplan : https://gcc.gnu.org/g:713613541254039a34e1dd8fd4a613a299af1fd6 commit r14-1615-g713613541254039a34e1dd8fd4a613a299af1fd6 Author: Alex Coplan Date: Tue Jun 6 11:04:45 2023 +0100 aarch64: Fix whitespace in ls64 builtin implementation [PR110100] The ls64 builtin code was using incorrect GNU style with eight spaces where there should be a tab. Fixed thusly. gcc/ChangeLog: PR target/110100 * config/aarch64/aarch64-builtins.cc (aarch64_init_ls64_builtins_types): Replace eight consecutive spaces with tabs. (aarch64_init_ls64_builtins): Likewise. (aarch64_expand_builtin_ls64): Likewise. * config/aarch64/aarch64.md (ld64b): Likewise. (st64b): Likewise. (st64bv): Likewise (st64bv0): Likewise.
[Bug target/110100] __builtin_aarch64_st64b stores to the wrong address
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110100 Alex Coplan changed: What|Removed |Added Status|NEW |ASSIGNED CC||acoplan at gcc dot gnu.org Assignee|unassigned at gcc dot gnu.org |acoplan at gcc dot gnu.org --- Comment #2 from Alex Coplan --- Testing a fix.
[Bug target/110100] __builtin_aarch64_st64b stores to the wrong address
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110100
Andrew Pinski changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Keywords||wrong-code
Ever confirmed|0 |1
Last reconfirmed||2023-06-02
--- Comment #1 from Andrew Pinski ---
Confirmed, you can get the similar wrong code with just:
#include
void do_st64b(int *a, data512_t data) {
__arm_st64b((void*)(a+512), data);
}
