[PATCH] D45772: [Minor patch] Fix IR Module Printing
Duncan P. N. Exon Smith via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 23 11:47:55 PDT 2018
dexonsmith added a comment.
Another high-level note (besides adding tests): please resubmit the patch with full context (e.g., `git diff -U9999999 HEAD^..`).
In https://reviews.llvm.org/D45772#1075420, @tyb0807 wrote:
> What would be the best way to fix this?
I suspect you can change the appropriate callers of printFunction to add a newline in between functions. E.g., something like this would work:
bool IsFirst = true;
for (auto &F : functions()) {
if (IsFirst)
IsFirst = false;
else
OS << "\n";
printFunction(F);
}
Repository:
rL LLVM
https://reviews.llvm.org/D45772
More information about the llvm-commits
mailing list