[PATCH] D105884: [llvm-readobj] Display multiple function names for stack size entries

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 23 01:43:54 PDT 2021


jhenderson accepted this revision.
jhenderson added a comment.

LGTM, with one nit.



================
Comment at: llvm/tools/llvm-readobj/ELFDumper.cpp:65
 #include <memory>
+#include <numeric>
 #include <string>
----------------
gbreynoo wrote:
> jhenderson wrote:
> > What's this for?
> This is for use of std::accumulate
I believe you can now remove this include?


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

https://reviews.llvm.org/D105884



More information about the llvm-commits mailing list