[PATCH] D36860: [Driver] Recognize DevDiv internal builds of MSVC, with a different directory structure
Stephan T. Lavavej via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 31 20:08:37 PDT 2017
[STL]
> I can't check in copies of `cl.exe` and `link.exe` (the former's version is inspected, so it can't just be a dummy file).
> I'm happy without upstream tests for this logic. In the long run, I hope that we can eliminate this internal directory structure and make it identical to the released directory structure.
[David Blaikie]
> Version as in file metadata, or version as in "cl -v" printed output or the like?
> If it's file metadata (such that Clang can discover it without having to run the binary on a test machine) might be worth making a dummy file with just the version metadata in it.
> But yeah *shrug* your call
Looks like it's just the metadata, so a dummy file would be possible (Don pointed out how the version metadata is created - I thought it was some complicated process, but it's not).
However, I'm not planning to write tests at the moment - I see that it is possible but there's a big leap from that to knowing how to actually write them, and I have to go back to working on C++17 features. If the logic for DevDiv-internal paths breaks, I'll deal with it then.
Thanks,
STL
More information about the cfe-commits
mailing list