diff options
| author | Roman Lebedev <lebedev.ri@gmail.com> | 2020-07-05 00:48:05 +0300 |
|---|---|---|
| committer | Roman Lebedev <lebedev.ri@gmail.com> | 2020-07-05 00:52:42 +0300 |
| commit | fbbb6884e108419692a88e28eeeaa92cfbc08122 (patch) | |
| tree | a6412b9b58c986be5c5b6727f9f35ba7afffb645 /clang/lib/Frontend/CompilerInvocation.cpp | |
| parent | 76123d338dc542d25cc9c4f20ddc49df1b9712cd (diff) | |
| download | llvm-fbbb6884e108419692a88e28eeeaa92cfbc08122.zip llvm-fbbb6884e108419692a88e28eeeaa92cfbc08122.tar.gz llvm-fbbb6884e108419692a88e28eeeaa92cfbc08122.tar.bz2 | |
[llvm-reduce] extractArgumentsFromModule(): don't crash when deleting instr twice
As it can be seen in newly-added (previously-crashing) test-case,
there can be a situation where multiple arguments are used in instr,
and we would schedule the same instruction to be deleted several times,
crashing when trying to delete it the second time.
We could either store WeakVH (done here), or use something set-like.
I think using WeakVH is prevalent in these cases elsewhere.
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions