[PATCH] D64165: [llvm-objdump] Keep warning for --disassemble-functions in correct order.

[Yuanfang Chen via Phabricator via llvm-commits]

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


================
Comment at: llvm/tools/llvm-objdump/llvm-objdump.cpp:384
 
-void warn(Twine Message) {
-  WithColor::warning(errs(), ToolName) << Message << "\n";
+static void warn(Twine Message, StringRef Prefix = "") {
+  // Output order between errs() and outs() matters especially for archive
----------------
ychen wrote:
> MaskRay wrote:
> > Why was `StringRef Prefix` added? It does not seem to be used.
> It was intended for the patch for PR35351. I've removed it. 
I meant to say PR41911.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64165