[PATCH] D55352: [elfabi] Introduce tool for ELF TextAPI

Armando Montanez via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 10 12:33:51 PST 2018


amontanez updated this revision to Diff 177586.
amontanez marked 15 inline comments as done.
amontanez added a comment.

Changed:

- Classes removed, functions are all standalone with no state.
- Now using Expected<std::unique_ptr<ELFStub>> for everything that would previously return a std::unique_ptr<ELFStub>.
- Added tests to check for proper failure output.

Removed:

- Automatic DT_SONAME generation.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55352

Files:
  llvm/test/CMakeLists.txt
  llvm/test/tools/llvm-elfabi/binary-read-arch.test
  llvm/test/tools/llvm-elfabi/fail-file-open.test
  llvm/test/tools/llvm-elfabi/read-unsupported-file.test
  llvm/test/tools/llvm-elfabi/replace-soname-tbe.test
  llvm/test/tools/llvm-elfabi/tbe-read-basic.test
  llvm/tools/LLVMBuild.txt
  llvm/tools/llvm-elfabi/CMakeLists.txt
  llvm/tools/llvm-elfabi/ELFObjHandler.cpp
  llvm/tools/llvm-elfabi/ELFObjHandler.h
  llvm/tools/llvm-elfabi/LLVMBuild.txt
  llvm/tools/llvm-elfabi/llvm-elfabi.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55352.177586.patch
Type: text/x-patch
Size: 15143 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181210/2d92a988/attachment.bin>


More information about the llvm-commits mailing list