aboutsummaryrefslogtreecommitdiff
path: root/gcc
diff options
context:
space:
mode:
Diffstat (limited to 'gcc')
-rw-r--r--gcc/analyzer/constraint-manager.cc10
-rw-r--r--gcc/testsuite/gcc.dg/analyzer/pr96650-1-notrans.c30
-rw-r--r--gcc/testsuite/gcc.dg/analyzer/pr96650-1-trans.c30
-rw-r--r--gcc/testsuite/gcc.dg/analyzer/pr96650-2-notrans.c30
-rw-r--r--gcc/testsuite/gcc.dg/analyzer/pr96650-2-trans.c30
5 files changed, 129 insertions, 1 deletions
diff --git a/gcc/analyzer/constraint-manager.cc b/gcc/analyzer/constraint-manager.cc
index e578e05..521501f 100644
--- a/gcc/analyzer/constraint-manager.cc
+++ b/gcc/analyzer/constraint-manager.cc
@@ -1752,7 +1752,15 @@ public:
if (m_cm_b->eval_condition (lhs, code, rhs).is_true ())
{
bool sat = m_out->add_constraint (lhs, code, rhs);
- gcc_assert (sat);
+ if (!sat)
+ {
+ /* If -fanalyzer-transitivity is off, we can encounter cases
+ where at least one of the two constraint_managers being merged
+ is infeasible, but we only discover that infeasibility
+ during merging (PR analyzer/96650).
+ Silently drop such constraints. */
+ gcc_assert (!flag_analyzer_transitivity);
+ }
}
}
diff --git a/gcc/testsuite/gcc.dg/analyzer/pr96650-1-notrans.c b/gcc/testsuite/gcc.dg/analyzer/pr96650-1-notrans.c
new file mode 100644
index 0000000..94c7555
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/analyzer/pr96650-1-notrans.c
@@ -0,0 +1,30 @@
+/* { dg-additional-options "-O2 -fno-analyzer-transitivity" } */
+
+int *wf;
+
+void
+yd (void);
+
+int
+cy (void);
+
+int *
+ee (int hp)
+{
+ if (hp != 0)
+ yd ();
+
+ return 0;
+}
+
+void
+z0 (int co)
+{
+ int l4 = sizeof (int);
+
+ aq:
+ wf = ee (l4);
+ if (l4 < co)
+ l4 = cy () + sizeof (int);
+ goto aq;
+}
diff --git a/gcc/testsuite/gcc.dg/analyzer/pr96650-1-trans.c b/gcc/testsuite/gcc.dg/analyzer/pr96650-1-trans.c
new file mode 100644
index 0000000..b20630b
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/analyzer/pr96650-1-trans.c
@@ -0,0 +1,30 @@
+/* { dg-additional-options "-O2 -fanalyzer-transitivity" } */
+
+int *wf;
+
+void
+yd (void);
+
+int
+cy (void);
+
+int *
+ee (int hp)
+{
+ if (hp != 0)
+ yd ();
+
+ return 0;
+}
+
+void
+z0 (int co)
+{
+ int l4 = sizeof (int);
+
+ aq:
+ wf = ee (l4);
+ if (l4 < co)
+ l4 = cy () + sizeof (int);
+ goto aq;
+}
diff --git a/gcc/testsuite/gcc.dg/analyzer/pr96650-2-notrans.c b/gcc/testsuite/gcc.dg/analyzer/pr96650-2-notrans.c
new file mode 100644
index 0000000..fc7c045
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/analyzer/pr96650-2-notrans.c
@@ -0,0 +1,30 @@
+/* { dg-additional-options "-fno-analyzer-transitivity" } */
+
+#include "analyzer-decls.h"
+
+int foo (void);
+
+/* Infeasible path, requiring transitivity to find. */
+
+void test_1 (int co, int y)
+{
+ if (4 < co)
+ if (co < y)
+ if (y == 0)
+ __analyzer_dump_path (); /* { dg-message "path" } */
+}
+
+/* Infeasible path, requiring transitivity to find, with a merger. */
+
+void test_2 (int co, int y, int z)
+{
+ if (4 < co)
+ if (co < y)
+ if (y == 0)
+ {
+ while (foo ())
+ {
+ }
+ __analyzer_dump_path (); /* { dg-message "path" } */
+ }
+}
diff --git a/gcc/testsuite/gcc.dg/analyzer/pr96650-2-trans.c b/gcc/testsuite/gcc.dg/analyzer/pr96650-2-trans.c
new file mode 100644
index 0000000..8d0c295
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/analyzer/pr96650-2-trans.c
@@ -0,0 +1,30 @@
+/* { dg-additional-options "-fanalyzer-transitivity" } */
+
+#include "analyzer-decls.h"
+
+int foo (void);
+
+/* Infeasible path, requiring transitivity to find. */
+
+void test_1 (int co, int y)
+{
+ if (4 < co)
+ if (co < y)
+ if (y == 0)
+ __analyzer_dump_path (); /* { dg-bogus "path" } */
+}
+
+/* Infeasible path, requiring transitivity to find, with a merger. */
+
+void test_2 (int co, int y, int z)
+{
+ if (4 < co)
+ if (co < y)
+ if (y == 0)
+ {
+ while (foo ())
+ {
+ }
+ __analyzer_dump_path (); /* { dg-bogus "path" } */
+ }
+}