[PATCH] D28365: [Driver] Updated for Visual Studio 2017

[Adrian McCarthy via Phabricator via cfe-commits]

amccarth added a comment.

(I could be missing something because the diff doesn't have much context.)

If I'm reading this right, it looks like it will no longer search the PATH in order to find cl.exe.  If that's the case, then I'm mildly worried about this change in behavior.  I know I have multiple versions of VS installed, and commonly select one just by moving its bin directory to the head of PATH.  The old behavior would find that one (which is used for things like defaulting -fms-compatibility-version) while the new behavior will find the newest one installed on the machine.

That's not necessarily a deal breaker, but it's an important change for Windows developers to be aware of.


https://reviews.llvm.org/D28365