[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