[PATCH] D85171: [Polly] Use containers with deterministic iteration order

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: polly/include/polly/CodeGen/BlockGenerators.h:37
   ///@{
-  using AllocaMapTy = DenseMap<const ScopArrayInfo *, AssertingVH<AllocaInst>>;
+  using AllocaMapTy = MapVector<const ScopArrayInfo *, AssertingVH<AllocaInst>>;
 
----------------
Where do we iterate over an AllocaMapTy?


================
Comment at: polly/include/polly/CodeGen/IslNodeBuilder.h:291
   /// An invariant load is identified by its pointer (the SCEV) and its type.
-  SmallSet<std::pair<const SCEV *, Type *>, 16> PreloadedPtrs;
+  SmallSetVector<std::pair<const SCEV *, Type *>, 16> PreloadedPtrs;
 
----------------
Where do we iterate over PreloadedPtrs?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85171