[gcc(refs/users/marxin/heads/marxin-gcc-benchmark-branch)] PR tree-optimization/92128 - fold more non-constant strlen relational expressions
Martin Liska
marxin@gcc.gnu.org
Mon Mar 30 10:13:13 GMT 2020
https://gcc.gnu.org/g:ccf86d54cb02ed24bc4568bd9fffdcdbf0bf68a8
commit ccf86d54cb02ed24bc4568bd9fffdcdbf0bf68a8
Author: Martin Sebor <msebor@redhat.com>
Date: Wed Feb 19 16:54:50 2020 -0700
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.
Diff:
---
gcc/testsuite/ChangeLog | 6 ++++++
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 ec3c34a2a30..049c343fe43 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 95ac29aa71f..f493514cfcc 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);
More information about the Gcc-cvs
mailing list