[PATCH] D69337: [PhiValues] Remove redundant searches in DenseMap

Ehud Katz via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 17 03:44:01 PST 2019


ekatz marked an inline comment as done.
ekatz added inline comments.


================
Comment at: llvm/lib/Analysis/PhiValues.cpp:87
       Reachable.insert(ComponentPhi);
-      DepthMap[ComponentPhi] = DepthNumber;
       for (Value *Op : ComponentPhi->incoming_values()) {
----------------
john.brawn wrote:
> It looks like this line is deleted and there's no equivalent to it in the patch.
It is done at line 124 of the patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69337





More information about the llvm-commits mailing list