diff options
author | Martin Sebor <msebor@redhat.com> | 2020-02-19 16:54:50 -0700 |
---|---|---|
committer | Martin Sebor <msebor@redhat.com> | 2020-02-19 16:54:50 -0700 |
commit | ccf86d54cb02ed24bc4568bd9fffdcdbf0bf68a8 (patch) | |
tree | 47b029527efaa4581bafbea4f7968bd04bf2f2ea | |
parent | 38c7b74d2e9007677de9c931725c03133ffd3b89 (diff) | |
download | gcc-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/ChangeLog | 6 | ||||
-rw-r--r-- | gcc/testsuite/gcc.dg/strlenopt-81.c | 13 |
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); |