[PATCH] D117121: [llvm] Remove uses of `std::vector<bool>`

[Duncan P. N. Exon Smith via Phabricator via llvm-commits]

dexonsmith added a comment.

Might be nice to stage a couple of these in whichever patch allows them to switch over, to make it clear why those APIs are needed/useful.



================
Comment at: llvm/lib/Transforms/Utils/SampleProfileInference.cpp:313
     // Find blocks that are reachable from the source
-    auto Visited = std::vector<bool>(NumBlocks(), false);
+    auto Visited = BitVector(NumBlocks(), false);
     findReachable(Func.Entry, Visited);
----------------
It doesn't look like this depends on any of the other in-flight patches; if so, it'd be nice to land this (and others like it) outside of this long queue since they don't need all the pieces to land.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117121