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

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 27 23:40:25 PDT 2017


mkazantsev added a comment.

Not obvious it is a win or not: we will have to check isDone() between all pushes, and only save pushing remaining operands for the last SCEV. If we traverse through a big graph where we will never find what we was looking for, we will only do extra checks without benefits.


https://reviews.llvm.org/D35931





More information about the llvm-commits mailing list