From: David Malcolm Date: Thu, 9 Jan 2020 15:12:59 +0000 (-0500) Subject: analyzer: fix ICE on METHOD_TYPE (PR 93212) X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=32077b693df8e3ed0424031a322df23822bf2f7e;p=gcc.git analyzer: fix ICE on METHOD_TYPE (PR 93212) PR analyzer/93212 reports an ICE when attempting to use -fanalyzer on a C++ source file. That isn't supported yet, but the fix is trivial (handling METHOD_TYPE as well as FUNCTION_TYPE). gcc/analyzer/ChangeLog: PR analyzer/93212 * region-model.cc (make_region_for_type): Use FUNC_OR_METHOD_TYPE_P rather than comparing against FUNCTION_TYPE. * region-model.h (function_region::function_region): Likewise. --- diff --git a/gcc/analyzer/ChangeLog b/gcc/analyzer/ChangeLog index 7d3b109944e..340288c08d4 100644 --- a/gcc/analyzer/ChangeLog +++ b/gcc/analyzer/ChangeLog @@ -1,3 +1,10 @@ +2020-01-14 David Malcolm + + PR analyzer/93212 + * region-model.cc (make_region_for_type): Use + FUNC_OR_METHOD_TYPE_P rather than comparing against FUNCTION_TYPE. + * region-model.h (function_region::function_region): Likewise. + 2020-01-14 David Malcolm * program-state.cc (sm_state_map::clone_with_remapping): Copy diff --git a/gcc/analyzer/region-model.cc b/gcc/analyzer/region-model.cc index b492222d64d..5c39be4fd7f 100644 --- a/gcc/analyzer/region-model.cc +++ b/gcc/analyzer/region-model.cc @@ -5963,7 +5963,7 @@ make_region_for_type (region_id parent_rid, tree type) if (TREE_CODE (type) == UNION_TYPE) return new union_region (parent_rid, type); - if (TREE_CODE (type) == FUNCTION_TYPE) + if (FUNC_OR_METHOD_TYPE_P (type)) return new function_region (parent_rid, type); /* If we have a void *, make a new symbolic region. */ diff --git a/gcc/analyzer/region-model.h b/gcc/analyzer/region-model.h index 2d0c0618bd0..eea808e61eb 100644 --- a/gcc/analyzer/region-model.h +++ b/gcc/analyzer/region-model.h @@ -1202,7 +1202,7 @@ public: function_region (region_id parent_rid, tree type) : map_region (parent_rid, type) { - gcc_assert (TREE_CODE (type) == FUNCTION_TYPE); + gcc_assert (FUNC_OR_METHOD_TYPE_P (type)); } function_region (const function_region &other) : map_region (other)