[PATCH] D76078: [AArch64][SVE] Add a pass for SVE intrinsic optimisations

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 24 11:16:58 PDT 2020


efriedma added inline comments.


================
Comment at: llvm/lib/Target/AArch64/SVEIntrinsicOpts.cpp:233
+  bool Changed = false;
+  for (auto *F : Functions) {
+    DominatorTree *DT = &getAnalysis<DominatorTreeWrapperPass>(*F).getDomTree();
----------------
Iterating over a SmallPtrSet is non-deterministic.

In this context, probably SetVector is the right data structure.


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

https://reviews.llvm.org/D76078





More information about the llvm-commits mailing list