[PATCH] D36860: [Driver] Recognize DevDiv internal builds of MSVC, with a different directory structure

don hinton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 18 07:36:14 PDT 2017


hintonda added a comment.

Looks good, but I'll defer to the owners.

Minor nit...



================
Comment at: lib/Driver/ToolChains/MSVC.cpp:142
           return true;
+        } else if (llvm::sys::path::filename(ParentPath) == "x86ret"
+            || llvm::sys::path::filename(ParentPath) == "x86chk"
----------------
Perhaps remove the else and grab the subdir name only once before doing comparisons.


https://reviews.llvm.org/D36860





More information about the cfe-commits mailing list