tree 60c4a45d138049a2ed28405ffecca849cc4be36b
parent fbbb6884e108419692a88e28eeeaa92cfbc08122
author Roman Lebedev <lebedev.ri@gmail.com> 1593900106 +0300
committer Roman Lebedev <lebedev.ri@gmail.com> 1593900106 +0300
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iQIzBAABCgAdFiEEjkF6151RK40WXe2HCDw+u0oWieAFAl8A/EoACgkQCDw+u0oW
 ieCE/w//UK2WlBd5TXj0DITSig7gxTzB83jaJhCUwrQG9ntuJV6LPk+Bun5y+YB5
 gsI4HI69QMlq6lISI1S86ioFn72puRi2QK3mns+XhodQx9UEEJF5cLoPxWtFVocq
 bliqfw57jZzCr+QnzYkhIAMLp4tO1rWNERazEmctEgv5pzX8gTPE8pe/zeIf0rZ4
 RpuF/47fcnacj0BE0k5xGwdUZbNX7PnQoUotJ4RSHjT/50VnLoUhor5qbf7OnWfK
 fYKoo10u/1zSf8rHGgfcBrqMy5cJc3Kk9NunPPjY5gbnkOrRBsWjSgGrb7qKQmrG
 wuyUlTP4VzFfWJMCte5KAkUvLLbE9bZ0R65xOSIiLrXdj3TfKxVj/qjCS4OlvANr
 NrU1yCYJaSgsdEiamEAjDRUKHWKlYJ9JOH+S06poP4j7YrLXUGT4G6kL+MusOQ5z
 BF5vjTdttReeD33q8Ypvt9EM1GA9dn7IaCIkWQSr//iPhtW7J8BGDmDemCf41x9N
 q0BuulyRCzHedN8+LJEN77RSNbqfh8Qt0uEFtp44iPjgxxokIc9VPZw3OzEv8Ipw
 mzwqtDdCWdx5RKrREPBIjEpqHHfi9Ey4/Cjg+nUav+CfIK02NGZOZ9IUDd6d0u+P
 8pokFM46ih83p/HcDxLsPUnwQjUX7hcUaz1Ze9NAzDLfaF2263Q=
 =UNpm
 -----END PGP SIGNATURE-----

[llvm-reduce] extractGVsFromModule(): 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 GV's 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.