[PATCH] D66418: [llvm-objdump] - Cleanup the error reporting.
George Rimar via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 19 07:37:13 PDT 2019
grimar marked an inline comment as done.
grimar added inline comments.
================
Comment at: tools/llvm-objdump/llvm-objdump.h:130
StringRef ArchitectureName = StringRef());
-void warn(Twine Message);
-LLVM_ATTRIBUTE_NORETURN void error(Twine Message);
LLVM_ATTRIBUTE_NORETURN void report_error(StringRef File, Twine Message);
LLVM_ATTRIBUTE_NORETURN void report_error(Error E, StringRef File);
----------------
btw, to be consistent, this should be `void report_error(Twine Message, StringRef File)`
But I'd like to do this in a separate NFC commit.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66418/new/
https://reviews.llvm.org/D66418
More information about the llvm-commits
mailing list