analyzer: fix ICE on unhandled tree codes in get_rvalue_1 [PR96641]
authorDavid Malcolm <dmalcolm@redhat.com>
Mon, 17 Aug 2020 22:18:00 +0000 (18:18 -0400)
committerDavid Malcolm <dmalcolm@redhat.com>
Tue, 18 Aug 2020 00:46:49 +0000 (20:46 -0400)
The old implementation of region_model::get_rvalue_1 gracefully handled
tree codes it didn't understand, returning "UNKNOWN", whereas the new
implementation (r11-2694-g808f4dfeb3a95f50f15e71148e5c1067f90a126d) had
an assertion left over from development, leading to ICEs.

This patch restores the old behavior for these cases.

gcc/analyzer/ChangeLog:
PR analyzer/96641
* region-model.cc (region_model::get_rvalue_1): Handle
unrecognized tree codes by returning "UNKNOWN.

gcc/testsuite/ChangeLog:
PR analyzer/96641
* g++.dg/analyzer/pr96641.C: New test.

gcc/analyzer/region-model.cc
gcc/testsuite/g++.dg/analyzer/pr96641.C [new file with mode: 0644]

index cd74c0f..305e964 100644 (file)
@@ -1097,7 +1097,7 @@ region_model::get_rvalue_1 (path_var pv, region_model_context *ctxt)
   switch (TREE_CODE (pv.m_tree))
     {
     default:
-      gcc_unreachable ();
+      return m_mgr->get_or_create_unknown_svalue (TREE_TYPE (pv.m_tree));
 
     case ADDR_EXPR:
       {
diff --git a/gcc/testsuite/g++.dg/analyzer/pr96641.C b/gcc/testsuite/g++.dg/analyzer/pr96641.C
new file mode 100644 (file)
index 0000000..eb11c85
--- /dev/null
@@ -0,0 +1,18 @@
+struct uh {
+  virtual void
+  sx ();
+};
+
+struct iz : uh {
+  virtual void
+  sx ()
+  {
+    sx ();
+  }
+};
+
+void
+a2 ()
+{
+  iz ().sx ();
+}