[PATCH] D64601: [MemorySSA] Use SetVector to avoid nondeterminism.

[Alina Sbirlea via Phabricator via llvm-commits]

asbirlea added a comment.

> You can use SetVector's "getArrayRef" and pass it as "void removeBlocks(ArrayRef<BasicBlock *> DeadBlocks);"

`removeBlocks` is querying DeadBlocks with `count(...)`. I can change the code to walk the Vector/ArrayRef, but it defeats the purpose of building a set.
It may be ok to change all callsites to just use vector too, assuming the  normal expected size is small, but it may be costly for pathological cases (large number of deadblocks).
If folks have a preference either way, please let me know.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64601/new/

https://reviews.llvm.org/D64601