diff options
| author | Nirav Dave <niravd@google.com> | 2017-06-09 12:57:35 +0000 |
|---|---|---|
| committer | Nirav Dave <niravd@google.com> | 2017-06-09 12:57:35 +0000 |
| commit | 43a4d8122f4210fb0603ccc08346bcb95cacf938 (patch) | |
| tree | af4756a779e33ad8e117337a02254ef24b5e374f /llvm/lib/DebugInfo/CodeView/DebugLinesSubsection.cpp | |
| parent | 1840901a2ff8493ca60579298be97eda8fadbcec (diff) | |
| download | llvm-43a4d8122f4210fb0603ccc08346bcb95cacf938.zip llvm-43a4d8122f4210fb0603ccc08346bcb95cacf938.tar.gz llvm-43a4d8122f4210fb0603ccc08346bcb95cacf938.tar.bz2 | |
Prevent RemoveDeadNodes from deleted already deleted node.
This prevents against assertion errors like PR32659 which occur from a
replacement deleting a node after it's been added to the list argument
of RemoveDeadNodes. The specific failure from PR32659 does not
currently happen, but it is still potentially possible. The underlying
cause is that the callers of the change dfunction builds up a list of
nodes to delete after having moved their uses and it possible that a
move of a later node will cause a previously deleted nodes to be
deleted.
Reviewers: bkramer, spatel, davide
Reviewed By: spatel
Subscribers: llvm-commits
Differential Revision: https://reviews.llvm.org/D33731
llvm-svn: 305070
Diffstat (limited to 'llvm/lib/DebugInfo/CodeView/DebugLinesSubsection.cpp')
0 files changed, 0 insertions, 0 deletions