[PATCH] D60974: Clang IFSO driver action.

Puyan Lotfi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 10 14:01:04 PDT 2019


plotfi added a comment.

In D60974#1498240 <https://reviews.llvm.org/D60974#1498240>, @jakehehrlich wrote:

> This shouldn't emit the .tbe format at all, the .tbe format is meant to be in near 1-1 correspondence with .so files, not with .o files. It has things like DT_NEEDED, and DT_SONAME related information that don't make sense for .o files for instance.
>
> If we put things under an --experimental flags, minimize what we add in the first patch, and try and make each additional field added need based, I'm ok with whatever being added to Clang assuming it doesn't interfere with other parts of the compiler.


So my latest diff addresses this. The schema resembles the tbe format but it is marked with experimental and will not load with the current llvm-elfabi tool. Should generate something resembling:

  --- !experimental-tapi-elf-v1
  Arch: x86_64
  Symbols:
    __Z16foo_default_visiii: { Type: Func }
    __Z6fvih_1ii: { Type: Func }
    __ZZ4fvihvE8fortytwo: { Type: Object, Size: 4 }
    __Z12someWeakFuncv: { Type: Func, Weak: true }
  ...

How does this look to you?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60974





More information about the llvm-commits mailing list