[PATCH] D78883: [VPlan] Move emission of \\l\"+\n to dumpBasicBlock (NFC).

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 14 05:20:32 PDT 2020


fhahn added a comment.

In D78883#2028543 <https://reviews.llvm.org/D78883#2028543>, @gilr wrote:

> Thanks for looking into this, Florian (and sorry for taking so long).
>  LGTM, except VPInterleaveRecipe::print() is missing here.
>  The patch stands by itself as a standardization of printing, similar to how the multi-line Function and Module print. If meant as a first step, would be good to elaborate on the full picture.


Thank you very much Gil! I've committed the patch with VPInterleaveRecipe::print()  updated as well and a more detailed commit message.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78883





More information about the llvm-commits mailing list