[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