[PATCH] D53051: [llvm-tapi] initial commit, supports reading ELF

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added a comment.

I'd like to start with a high level comment too. I wonder if your final vision includes adding native support of ELF TAPI to lld. As I expressed in the mailing list threads for the proposal, honestly, I'm not still convinced that TAPI for ELF is a good idea, considering the amount of overlap that the regular ELF and ELF TAPI can do and the cost of adding a completely new format. A lot of things you can do with ELF TAPI can also be done with the regular ELF format, and looks like other benefits you would get with ELF TAPI be achieved with more lightweight approach. I think I cannot support adding ELF TAPI to lld if you have a plan to do so. To convince me, can you please give me examples that ELF TAPI is absolutely needed/useful?


Repository:
  rL LLVM

https://reviews.llvm.org/D53051