[PATCH] D60974: Clang IFSO driver action.

[Saleem Abdulrasool via Phabricator via cfe-commits]

compnerd added a comment.

The strong motivation here is to avoid having the list be maintained outside of the source code.  I am not too strongly tied to the YAML approach as long as the binaries that we get at the end are equivalent to the pruned ELF binaries from the YAML2ELF approach.  However, the trade off here is valuable - it allows for the source code to be the source of truth for the interface set.  The intermediate emission is due to the fact that this requires program level visibility and having the intermediates an extra step to merge is relatively simpler.  In fact, you could emit the TBE files from this and check that into the tree if you really wanted (but allowing the source to remain the source of truth I think is a valid and valuable trade off).  The reason for the selection of YAML is simply pragmatism - that is already available in the tree.  If the TBE linker is available now, I have no issue with that being the intermediate representation (assuming that the representation is easy to parse programatically).


Repository:
  rC Clang

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

https://reviews.llvm.org/D60974