[PATCH] D92380: [llvm-readelf/obj] - Refine the error message about the broken string table.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 1 18:32:07 PST 2020


MaskRay accepted this revision.
MaskRay added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/tools/llvm-readobj/ELFDumper.cpp:2047
           else
-            reportWarning(E.takeError(), ObjF.getFileName());
+            reportUniqueWarning("unable to get the string table for the " +
+                                describe(Sec) + ": " + toString(E.takeError()));
----------------
Other messages don't prepend `the`


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

https://reviews.llvm.org/D92380



More information about the llvm-commits mailing list