[PATCH] D71957: [llvm-size] print a blank line between archieve members when using sysv format

[Xuanda Yang via Phabricator via llvm-commits]

TH3CHARLie marked 2 inline comments as done.
TH3CHARLie added inline comments.


================
Comment at: llvm/tools/llvm-size/llvm-size.cpp:449
     outs() << format(fmt.str().c_str(), static_cast<const char *>("Total"),
-                     total);
+                     total) << "\n";
   } else {
----------------
jhenderson wrote:
> Jim wrote:
> > Put "<< "\n"" in the new line look better.
> > 
> > ```
> > outs() << format... 
> >        << "\n";
> > ```
> I think this is correct, but to be sure if you haven't already, run git-clang-format on the modified part to verify your formatting choices match the official style (see https://llvm.org/docs/GettingStarted.html#sending-patches).
Thanks for the heads up and I've run the clang-format which put the "\n\n" into a newline


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71957