arphaman added a comment.

In D105257#2853438 <https://reviews.llvm.org/D105257#2853438>, @dexonsmith 
wrote:

> Could the DarwinSDKInfo changes be tested directly with C++ unit tests? Since 
> this stuff is in Basic it'd be nice to test it separately from the driver. If 
> so, maybe they could also mostly land in a separate prep commit (except the 
> update to `clang::parseDarwinSDKInfo` which is what changes the Driver 
> behaviour).

I split the patch to also https://reviews.llvm.org/D105958 like you suggested.


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

https://reviews.llvm.org/D105257

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

Reply via email to