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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to