[PATCH] D54100: [LegacyPassManager] Add NL to -print-before/-print-after banner

Fedor Sergeev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 5 07:00:36 PST 2018


fedor.sergeev requested changes to this revision.
fedor.sergeev added a comment.
This revision now requires changes to proceed.

I believe this is not a right place for a fix.
Banner in some cases is just a prefix for a full banner line
(see PrintFunctionPass::run, handling for forcePrintModuleIR).

So the right way to fix is to add \n in a couple of places where it is still missing
(say, in PrintFunctionPass::run, else branch of forcePrintModuleIR condition).


Repository:
  rL LLVM

https://reviews.llvm.org/D54100





More information about the llvm-commits mailing list