[PATCH] D35931: [SCEV] Do not visit nodes twice in containsConstantSomewhere

Sanjoy Das via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 27 00:39:19 PDT 2017


sanjoy added inline comments.


================
Comment at: lib/Analysis/ScalarEvolution.cpp:2681
   SmallVector<const SCEV *, 4> Ops;
+  SmallSet<const SCEV *, 4> Visited;
   Ops.push_back(StartExpr);
----------------
Can you use `SCEVExprContains` here?


https://reviews.llvm.org/D35931





More information about the llvm-commits mailing list