[PR100518] store by mult pieces: keep addr in Pmode
Commit Message
The conversion of a MEM address to ptr_mode in
try_store_by_multiple_pieces was misguided: copy_addr_to_reg expects
Pmode for addresses.
Regstrapped on x86_64-linux-gnu, testcase verified with a cross to
aarch64. Ok to install?
for gcc/ChangeLog
PR target/100518
* builtins.c (try_store_by_multiple_pieces): Drop address
conversion to ptr_mode.
for gcc/testsuite/ChangeLog
PR target/100518
* gcc.target/aarch64/pr100518.c: New.
---
gcc/builtins.c | 2 +-
gcc/testsuite/gcc.target/aarch64/pr100518.c | 9 +++++++++
2 files changed, 10 insertions(+), 1 deletion(-)
create mode 100644 gcc/testsuite/gcc.target/aarch64/pr100518.c
Comments
On 12/9/2021 3:18 PM, Alexandre Oliva via Gcc-patches wrote:
> The conversion of a MEM address to ptr_mode in
> try_store_by_multiple_pieces was misguided: copy_addr_to_reg expects
> Pmode for addresses.
>
> Regstrapped on x86_64-linux-gnu, testcase verified with a cross to
> aarch64. Ok to install?
>
>
> for gcc/ChangeLog
>
> PR target/100518
> * builtins.c (try_store_by_multiple_pieces): Drop address
> conversion to ptr_mode.
>
> for gcc/testsuite/ChangeLog
>
> PR target/100518
> * gcc.target/aarch64/pr100518.c: New.
OK
jeff
@@ -4003,7 +4003,7 @@ try_store_by_multiple_pieces (rtx to, rtx len, unsigned int ctz_len,
constfundata = &valc;
}
- rtx ptr = copy_addr_to_reg (convert_to_mode (ptr_mode, XEXP (to, 0), 0));
+ rtx ptr = copy_addr_to_reg (XEXP (to, 0));
rtx rem = copy_to_mode_reg (ptr_mode, convert_to_mode (ptr_mode, len, 0));
to = replace_equiv_address (to, ptr);
set_mem_align (to, align);
new file mode 100644
@@ -0,0 +1,9 @@
+/* { dg-do compile } */
+/* { dg-options "-mabi=ilp32 -mstrict-align -O2" } */
+
+int unsigned_range_min, unsigned_range_max, a11___trans_tmp_1;
+
+void a11() {
+ a11___trans_tmp_1 = unsigned_range_max < unsigned_range_min;
+ __builtin_memset((char *)1, 0, a11___trans_tmp_1);
+}