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

Owen Reynolds via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 20 09:37:53 PDT 2021


gbreynoo marked 3 inline comments as done.
gbreynoo added inline comments.


================
Comment at: llvm/tools/llvm-readobj/ELFDumper.cpp:5829
+
+  OS << join(FuncNames.begin(), FuncNames.end(), ", ") << "\n";
 }
----------------
After some offline discussion around the inefficiency of the previous implementation, the use of join makes more sense here.


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

https://reviews.llvm.org/D105884



More information about the llvm-commits mailing list