[PATCH] D74533: [Float2Int] Use a type alias for the RootSet. NFC

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 14 17:07:13 PST 2020


craig.topper added inline comments.


================
Comment at: llvm/include/llvm/Transforms/Scalar/Float2Int.h:48
   MapVector<Instruction *, ConstantRange> SeenInsts;
-  SmallPtrSet<Instruction *, 8> Roots;
+  RootSet Roots;
   EquivalenceClasses<Instruction *> ECs;
----------------
Why is Roots both a member variable and passed to multiple member functions?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74533





More information about the llvm-commits mailing list