[PATCH] D60974: Clang IFSO driver action.

Puyan Lotfi via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 24 14:06:35 PDT 2019


plotfi added a comment.

In D60974#1477302 <https://reviews.llvm.org/D60974#1477302>, @phosek wrote:

> Aside from comments that @jakehehrlich already made which I agree with, I'm also concerned about the use of `yaml2obj`. AFAIK that tool has always been intended as a testing, not a production tool, but with this change this would no longer be the case. Specifically, it formalizes the `yaml2obj` input format by making it one of the output formats supported by Clang. I believe that this is such a fundamental change that it should be discussed separately before moving on with this implementation.


My main goal here is to emit some kind of a mergable format. If not yaml or tbe, then directly emitting the ELF. yaml is a stepping stone for now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60974





More information about the cfe-commits mailing list