[PATCH] D85257: [X86] Optimize getImpliedDisabledFeatures & getImpliedEnabledFeatures'

[Nick Desaulniers via Phabricator via llvm-commits]

nickdesaulniers added inline comments.


================
Comment at: llvm/lib/Support/X86TargetParser.cpp:101-104
+    for (unsigned I = 0, E = array_lengthof(Bits); I != E; ++I)
+      if (Bits[I] != RHS.Bits[I])
+        return false;
+    return true;
----------------
MaskRay wrote:
> nickdesaulniers wrote:
> > are these more concise to `return llvm::any_of` here and above?
> llvm::any_of is usually used on one collection. Here we are comparing two collections..
Sure for `operator !=`, but `operator bool` looks like one collection though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85257