[PATCH] D76375: [VPlan] Do not print mapping for Value2VPValue.
Ayal Zaks via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 18 13:35:22 PDT 2020
Ayal accepted this revision.
Ayal added a comment.
This revision is now accepted and ready to land.
Minor nit.
Thanks for following up quickly on https://reviews.llvm.org/D76200#inline-694794
================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.cpp:585
OS << ", where:";
if (Plan.BackedgeTakenCount) {
OS << "\\n";
----------------
This second `if` is now redundant.
Once we can assign VPValues (meaningful) names, this should be removed as well.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D76375/new/
https://reviews.llvm.org/D76375
More information about the llvm-commits
mailing list