[PATCH] D84240: [llvm-readelf] - Introduce describe() helper functions.

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 22 01:04:20 PDT 2020


jhenderson accepted this revision.
jhenderson added a comment.

Just a nit from me. Otherwise LGTM.



================
Comment at: llvm/tools/llvm-readobj/ELFDumper.cpp:382
   std::function<Error(const Twine &Msg)> WarningHandler;
-  void reportUniqueWarning(Error Err) const;
+  void reportUniqueWarning(Error Err) const;;
 };
----------------
Nit: accidental ';'


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

https://reviews.llvm.org/D84240





More information about the llvm-commits mailing list