tree a6412b9b58c986be5c5b6727f9f35ba7afffb645
parent 76123d338dc542d25cc9c4f20ddc49df1b9712cd
author Roman Lebedev <lebedev.ri@gmail.com> 1593899285 +0300
committer Roman Lebedev <lebedev.ri@gmail.com> 1593899562 +0300
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iQIzBAABCgAdFiEEjkF6151RK40WXe2HCDw+u0oWieAFAl8A+ioACgkQCDw+u0oW
 ieCizA//XQPFCm9AMLTUj72Ia5VSx6Ly3hh4Ynprz9Ua316uf6Y+0atf1ekizw6H
 V0v0fBlYnnnGIW6GxK5rSOIHk/IPKxO6lhM6+XBvy+5sGKyScX+KEQHy/zFA3CO5
 wwwTptpmnGZ8k42N1T9h4QxC3IYdevePVoA/4e/eorzWWwZGukIW4B8Ve1ikTQbc
 PynxGeGt8zOxqnTXgMiqNw2KMxdaWyVNUT7YeRR2bO/arEbn2b2cEaPUlRiAvxpm
 tLf7lGZ6xMUA82Qaaem2+S0kAn1u5w1KiztwpSuAnpEgppN/fJV25d1/rNYPo5my
 NmtkmuMQl+nbht3zaHoR8JL+4Uj/2j9SAr5bL1ebZlYWjvGke80U6G2HfESMQ+P2
 4z86GSKH545xiMm0D8U7Kv+qDGSy12Bw5D3pMvvFzzS03tHt+UkTkY4fMpOQyA1U
 bd0Tc57Uhl++ZKfKhFvQtlmX0vzBa2eetR09hzb++vuMdTgBvw0PuNK0N3CAWHyP
 oJ0oUFpew3ldpa2EacpXNduHhK89OgdeuCf/Mkl7Rw70PyiFtSpub12ASv1DeG8C
 bIq0cUkFsNIbAuavv9OMRRCbTZ1rwYYRtVqLx6kxhtpKV7GRcCYiRSmHmcbVJqfZ
 xdSzamXAM3w2Qx1WO9eKJnHQqW437zIxUQmFT8iPPbe+LX+75eg=
 =FKhn
 -----END PGP SIGNATURE-----

[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.