kristina added a comment.

Ping. Would like someone else to co-review this, everything looks fine aside 
from tests (I think a portion of this could use a short unit test with regards 
to creating the actual target instance). Also your FileCheck test is only for 
invoking `clang -cc1` from the looks, can you add coverage for static linker 
invocations (as driver is also generally used to invoke the linker, unless 
that's explicitly not the case on Hurd)?


Repository:
  rC Clang

https://reviews.llvm.org/D54379



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to