diff options
| author | Jakub Jelinek <jakub@gcc.gnu.org> | 2018-01-23 21:36:34 +0100 |
|---|---|---|
| committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2018-01-23 21:36:34 +0100 |
| commit | 508d0c3da24c91bdabf7fbd453b511bfccbb2958 (patch) | |
| tree | 783ff70265bd9ba0f304d4c634b318e3605f7b2d /gcc/cp/tree.c | |
| parent | 56dbd05b0ed265ec37f66234487aaad6e330c7ce (diff) | |
| download | gcc-508d0c3da24c91bdabf7fbd453b511bfccbb2958.zip gcc-508d0c3da24c91bdabf7fbd453b511bfccbb2958.tar.gz gcc-508d0c3da24c91bdabf7fbd453b511bfccbb2958.tar.bz2 | |
re PR sanitizer/83987 (ICE with OpenMP, sanitizer and virtual bases)
PR sanitizer/83987
* tree.c (cp_free_lang_data): Change DECL_VALUE_EXPR of
DECL_OMP_PRIVATIZED_MEMBER vars to error_mark_node.
* g++.dg/ubsan/pr83987.C: New test.
From-SVN: r256997
Diffstat (limited to 'gcc/cp/tree.c')
| -rw-r--r-- | gcc/cp/tree.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/gcc/cp/tree.c b/gcc/cp/tree.c index ed51c28..e87c596 100644 --- a/gcc/cp/tree.c +++ b/gcc/cp/tree.c @@ -5274,6 +5274,16 @@ cp_free_lang_data (tree t) /* We do not need the leftover chaining of namespaces from the binding level. */ DECL_CHAIN (t) = NULL_TREE; + /* Set DECL_VALUE_EXPRs of OpenMP privatized member artificial + decls to error_mark_node. These are DECL_IGNORED_P and after + OpenMP lowering they aren't useful anymore. Clearing DECL_VALUE_EXPR + doesn't work, as expansion could then consider them as something + to be expanded. */ + if (VAR_P (t) + && DECL_LANG_SPECIFIC (t) + && DECL_OMP_PRIVATIZED_MEMBER (t) + && DECL_IGNORED_P (t)) + SET_DECL_VALUE_EXPR (t, error_mark_node); } /* Stub for c-common. Please keep in sync with c-decl.c. |