MSVC: Use compiler driver for linking

To be able to use cpp.driverFlags and cpp.driverLinkerFlags with clang-
cl. This patchset makes possible to use clang-cl with "-
fsanitize=address" flag without passing the sanitizer libraries manually
to the linker

There's also a behavior change in which linker is used - clang-cl uses
native linker by default. Old behavior can be restored by setting
cpp.linkerVariant to "lld"

Fixes: QBS-1522
Change-Id: I9528ce40aa5fdfab987672b15fffd830fa2d6376
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>

4 files changed, 38 insertions, 0 deletions

diff --git a/tests/auto/blackbox/testdata/generate-linker-map-file/generate-linker-map-file.qbs b/tests/auto/blackbox/testdata/generate-linker-map-file/generate-linker-map-file.qbs
index 815e64853..8c971a747 100644
--- a/tests/auto/blackbox/testdata/generate-linker-map-file/generate-linker-map-file.qbs
+++ b/tests/auto/blackbox/testdata/generate-linker-map-file/generate-linker-map-file.qbs
@@ -2,11 +2,14 @@ Project {
     CppApplication {
         name: "app-map"
         files: ["main.cpp"]
+        // lld-link has different flag for map files, test it by switching to "lld" linkerVariant
+        Properties { condition: qbs.toolchain.contains("clang-cl"); cpp.linkerVariant: "lld" }
         cpp.generateLinkerMapFile: true
     }
     CppApplication {
         name: "app-nomap"
         files: ["main.cpp"]
+        Properties { condition: qbs.toolchain.contains("clang-cl"); cpp.linkerVariant: "lld" }
         cpp.generateLinkerMapFile: false
     }
     CppApplication {
diff --git a/tests/auto/blackbox/testdata/response-files/response-files.qbs b/tests/auto/blackbox/testdata/response-files/response-files.qbs
index fbb6f0518..1750ead3b 100644
--- a/tests/auto/blackbox/testdata/response-files/response-files.qbs
+++ b/tests/auto/blackbox/testdata/response-files/response-files.qbs
@@ -38,6 +38,9 @@ Project {
     Product {
         name: "lotsofobjects"
         type: ["dynamiclibrary"]
+        // clang-cl does not use response file internally, thus linker complains that command is
+        // too long. This can be worked around by calling the linker directly
+        cpp.linkerMode: qbs.toolchain.contains("clang-cl") ? "manual" : original
         Depends { name: "cpp" }
         Rule {
             multiplex: true
diff --git a/tests/auto/blackbox/testdata/sanitizer/sanitizer.cpp b/tests/auto/blackbox/testdata/sanitizer/sanitizer.cpp
new file mode 100644
index 000000000..4a7c3ee32
--- /dev/null
+++ b/tests/auto/blackbox/testdata/sanitizer/sanitizer.cpp
@@ -0,0 +1,4 @@
+int main(int argc, char *argv[])
+{
+    return 0;
+}
diff --git a/tests/auto/blackbox/testdata/sanitizer/sanitizer.qbs b/tests/auto/blackbox/testdata/sanitizer/sanitizer.qbs
new file mode 100644
index 000000000..0aa8ed964
--- /dev/null
+++ b/tests/auto/blackbox/testdata/sanitizer/sanitizer.qbs
@@ -0,0 +1,28 @@
+CppApplication {
+    property string sanitizer
+
+    property bool supportsSanitizer: {
+        if (qbs.toolchain.contains("clang-cl"))
+            // only these are supported
+            return sanitizer === "address" || sanitizer === "undefined";
+        if (!qbs.toolchain.contains("gcc"))
+            return false;
+        return true;
+    }
+
+    condition: {
+        if (!sanitizer)
+            return true;
+        if (!supportsSanitizer)
+            console.info("Compiler does not support sanitizer");
+        return supportsSanitizer;
+    }
+    qbs.buildVariant: "release"
+    cpp.cxxLanguageVersion: "c++11"
+    cpp.minimumMacosVersion: "10.8"
+    consoleApplication: true
+    cpp.runtimeLibrary: "static"
+    cpp.driverFlags: sanitizer ? ["-fsanitize=" + sanitizer] : []
+    cpp.debugInformation: true
+    files: "sanitizer.cpp"
+}