[PATCH] D74658: [mlir][NFC] Fix 'gatherLoops' utility

Andy Davis via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 18 12:44:50 PST 2020


andydavis1 accepted this revision.
andydavis1 added inline comments.


================
Comment at: mlir/test/lib/Transforms/TestAffineDataCopy.cpp:46
   // Gather all AffineForOps by loop depth.
-  DenseMap<unsigned, SmallVector<AffineForOp, 2>> depthToLoops;
+  SmallVector<SmallVector<AffineForOp, 2>, 8> depthToLoops;
   gatherLoops(getFunction(), depthToLoops);
----------------
Typically we don't see SmallVector<SmallVector.  These patterns are typically std::vector<SmallVector.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74658





More information about the llvm-commits mailing list