[PATCH] D60974: Clang IFSO driver action.

Petr Hosek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 24 11:01:41 PDT 2019


phosek added a comment.

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.


Repository:
  rC Clang

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

https://reviews.llvm.org/D60974





More information about the cfe-commits mailing list