[gcc(refs/users/marxin/heads/PR93800-ppc-alignment-v2)] Restore alignment in rs6000 target.
Martin Liska
marxin@gcc.gnu.org
Mon Mar 9 10:15:19 GMT 2020
https://gcc.gnu.org/g:d38186d6f649be4824653a89dfdcb01079f4cd4f
commit d38186d6f649be4824653a89dfdcb01079f4cd4f
Author: Martin Liska <mliska@suse.cz>
Date: Thu Mar 5 13:47:15 2020 +0100
Restore alignment in rs6000 target.
gcc/ChangeLog:
2020-03-05 Martin Liska <mliska@suse.cz>
PR target/93800
* config/rs6000/rs6000.c (rs6000_option_override_internal):
Remove set of str_align_loops and str_align_jumps as these
should be set in previous 2 conditions in the function.
gcc/testsuite/ChangeLog:
2020-03-09 Martin Liska <mliska@suse.cz>
PR target/93800
* gcc.target/powerpc/pr93800.c: New test.
Diff:
---
gcc/config/rs6000/rs6000.c | 5 -----
gcc/testsuite/gcc.target/powerpc/pr93800.c | 13 +++++++++++++
2 files changed, 13 insertions(+), 5 deletions(-)
diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c
index ecbf7ae0c59..848a4ef451e 100644
--- a/gcc/config/rs6000/rs6000.c
+++ b/gcc/config/rs6000/rs6000.c
@@ -4363,11 +4363,6 @@ rs6000_option_override_internal (bool global_init_p)
str_align_loops = "16";
}
}
-
- if (flag_align_jumps && !str_align_jumps)
- str_align_jumps = "16";
- if (flag_align_loops && !str_align_loops)
- str_align_loops = "16";
}
/* Arrange to save and restore machine status around nested functions. */
diff --git a/gcc/testsuite/gcc.target/powerpc/pr93800.c b/gcc/testsuite/gcc.target/powerpc/pr93800.c
new file mode 100644
index 00000000000..40ad7da3cac
--- /dev/null
+++ b/gcc/testsuite/gcc.target/powerpc/pr93800.c
@@ -0,0 +1,13 @@
+/* { dg-do compile } */
+/* { dg-options "-m32 -mcpu=860 -O2" } */
+/* { dg-final { scan-assembler-not "\\.p2align 4" } } */
+
+volatile int g;
+int f(int a, int b)
+{
+ int i;
+
+ for (i = 0; i < b; i++)
+ a += g;
+ return a;
+}
More information about the Gcc-cvs
mailing list