STL_MSFT marked an inline comment as done.
STL_MSFT added a comment.

In https://reviews.llvm.org/D36860#845469, @thakis wrote:

> This approach looks good to me.


Thanks!

> Is there any chance we could have a test for this?

Unfortunately, testing this involves having a VS toolset with the 
DevDiv-internal directory structure. Preparing that is a laborious manual 
process, and requires installation of the Windows SDK (I sent repro 
instructions to cfe-dev). Even given a real installation of VS, I don't think I 
could write automated test code to extract a copy with the necessary directory 
structure (doing so would almost involve replicating the codepaths being 
touched here). But now that I know that this code exists and that Clang 
occasionally changes it, especially in response to new VS directory structures, 
I'll watch for new betas/RCs and verify them against our internal builds.


https://reviews.llvm.org/D36860



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

Reply via email to