aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMartin Sebor <msebor@redhat.com>2020-02-19 16:54:50 -0700
committerMartin Sebor <msebor@redhat.com>2020-02-19 16:54:50 -0700
commitccf86d54cb02ed24bc4568bd9fffdcdbf0bf68a8 (patch)
tree47b029527efaa4581bafbea4f7968bd04bf2f2ea
parent38c7b74d2e9007677de9c931725c03133ffd3b89 (diff)
downloadgcc-ccf86d54cb02ed24bc4568bd9fffdcdbf0bf68a8.zip
gcc-ccf86d54cb02ed24bc4568bd9fffdcdbf0bf68a8.tar.gz
gcc-ccf86d54cb02ed24bc4568bd9fffdcdbf0bf68a8.tar.bz2
PR tree-optimization/92128 - fold more non-constant strlen relational expressions
gcc/testsuite/ChangeLog: * gcc.dg/strlenopt-81.c: Align arrays to let strictly aligned targets optimize away calls as expected.
-rw-r--r--gcc/testsuite/ChangeLog6
-rw-r--r--gcc/testsuite/gcc.dg/strlenopt-81.c13
2 files changed, 15 insertions, 4 deletions
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index ec3c34a..049c343 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,9 @@
+2020-02-19 Martin Sebor <msebor@redhat.com>
+
+ PR tree-optimization/92128
+ * gcc.dg/strlenopt-81.c: Align arrays to let strictly aligned targets
+ optimize away calls as expected.
+
2020-02-19 Uroš Bizjak <ubizjak@gmail.com>
* gcc.dg/gimplefe-41.c: Add -msse2 additional options for x86 targets.
diff --git a/gcc/testsuite/gcc.dg/strlenopt-81.c b/gcc/testsuite/gcc.dg/strlenopt-81.c
index 95ac29a..f493514 100644
--- a/gcc/testsuite/gcc.dg/strlenopt-81.c
+++ b/gcc/testsuite/gcc.dg/strlenopt-81.c
@@ -1,6 +1,6 @@
-/* PR tree-optimization/ - fold strlen relational expressions
+/* PR tree-optimization/91996 - fold non-constant strlen relational expressions
{ dg-do run }
- { dg-options "-O2 -Wall -Wno-unused-local-typedefs -fdump-tree-optimized" } */
+ { dg-options "-O2 -Wall -Wno-unused-local-typedefs" } */
typedef __SIZE_TYPE__ size_t;
@@ -17,7 +17,10 @@ typedef __SIZE_TYPE__ size_t;
CAT (CAT (test_on_line_, __LINE__), _not_eliminated)(); \
} typedef void dummy_type
-char a[32], b[32];
+
+/* Set the alignment for targets that depend on it in order to
+ optimize away the ELIM calls. See pr92128. */
+__attribute__ ((aligned(4))) char a[32], b[32];
void init (void)
{
@@ -89,7 +92,9 @@ NOIPA void test_local_cpy_4 (void)
size_t blen = __builtin_strlen (b);
if (blen < 9) return;
- char a[10] = "abcdefgh";
+/* Set the alignment for targets that depend on it in order to
+ optimize away the ELIM calls. See pr92128. */
+ __attribute__ ((aligned(4))) char a[10] = "abcdefgh";
char *d = a;
__builtin_memcpy (d, b, 4);