[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