[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