[PATCH] D16204: ValueTracking: Use fixed array for assumption exclude set in Query; NFC

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 19 12:18:19 PST 2016


sanjoy added inline comments.

================
Comment at: lib/Analysis/ValueTracking.cpp:117
@@ +116,3 @@
+  bool isExcluded(const Value *Value) const {
+    for (unsigned I = 0; I < NumExcluded; ++I) {
+      if (Excluded[I] == Value)
----------------
sanjoy wrote:
> This can be a call to `find` or `any_of` from `STLExtras.h` if you use an `std::array` for `Excluded`.
Sorry, just realized that it won't.


Repository:
  rL LLVM

http://reviews.llvm.org/D16204





More information about the llvm-commits mailing list