diff mbox series

ipa-sra: Testcase that removing a "returns_nonnull" retval works

Message ID ri6zgq4xwqc.fsf@suse.cz
State New
Headers show
Series ipa-sra: Testcase that removing a "returns_nonnull" retval works | expand

Commit Message

Martin Jambor Nov. 16, 2021, 9:42 a.m. UTC
Hi,

since we can now remove return values of functions with return_nonnull
type attribute, I'll feel a bit safer if we can test this does not ICE
when someone attempts to access a non-existent call LHS.  Eventually
we should probably drop the attribute when this happens.

Tested on x86_64-linux, I will push it to master momentarily.

Martin


gcc/testsuite/ChangeLog:

2021-11-15  Martin Jambor  <mjambor@suse.cz>

	* gcc.dg/ipa/ipa-sra-ret-nonull.c: New test.
---
 gcc/testsuite/gcc.dg/ipa/ipa-sra-ret-nonull.c | 40 +++++++++++++++++++
 1 file changed, 40 insertions(+)
 create mode 100644 gcc/testsuite/gcc.dg/ipa/ipa-sra-ret-nonull.c
diff mbox series

Patch

diff --git a/gcc/testsuite/gcc.dg/ipa/ipa-sra-ret-nonull.c b/gcc/testsuite/gcc.dg/ipa/ipa-sra-ret-nonull.c
new file mode 100644
index 00000000000..18c13efd609
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/ipa/ipa-sra-ret-nonull.c
@@ -0,0 +1,40 @@ 
+/* { dg-do compile } */
+/* { dg-options "-O2 -fdump-ipa-sra-details"  } */
+
+volatile void *gp;
+volatile void *gq;
+char buf[16];
+
+__attribute__((returns_nonnull, noinline))
+static char *
+foo (char *p, char *q)
+{
+  gq = q;
+  gp = p;
+  return q;
+}
+
+__attribute__((returns_nonnull, noinline))
+static char *
+bar (char *p, char *q)
+{
+  return foo (p, q) + 8;
+}
+
+__attribute__((noipa))
+static char *
+get_charp (void)
+{
+  return &buf[0];
+}
+
+int
+main ()
+{
+  char *r;
+  asm volatile ("" : : : "memory");
+  r = bar (get_charp (), get_charp ());
+  return 0;
+}
+
+/* { dg-final { scan-ipa-dump-times "Will SKIP return." 2 "sra" } } */