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 <mjam...@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 --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" } } */
--
2.33.0
