(C++ modules support) fix clang-cl (disable build.c++.modules.two_phases)

Author: Arthapz

tests/projects/c++/modules/hello_with_pch/test.lua

@@ -1 +1,9 @@
 inherit(".test_base")
+
+function main(_)
+    -- clang-cl doesn't support mixing pch and C++ module atm
+    local clang_options = {compiler = "clang", version = CLANG_MIN_VER, disable_clang_cl = true}
+    local gcc_options = {compiler = "gcc", version = GCC_MIN_VER}
+    local msvc_options = {version = MSVC_MIN_VER}
+    run_tests(clang_options, gcc_options, msvc_options)
+end

tests/projects/c++/modules/test_base.lua

@@ -4,6 +4,7 @@ import("core.tool.toolchain")
 import("utils.ci.is_running", {alias = "ci_is_running"})
 
 CLANG_MIN_VER = "17"
+CLANG_CL_MIN_VER = "19"
 GCC_MIN_VER = "11"
 MSVC_MIN_VER = "14.29"
 
@@ -127,9 +128,12 @@ function run_tests(clang_options, gcc_options, msvc_options)
         if clang_options then
             build_tests("llvm", clang_options)
             build_tests("clang", clang_options)
-            local clang_cl_options = table.clone(clang_options)
-            clang_cl_options.compiler = "clang-cl"
-            build_tests("clang-cl", clang_cl_options)
+            if not clang_options.disable_clang_cl then
+                local clang_cl_options = table.clone(clang_options)
+                clang_cl_options.compiler = "clang-cl"
+                clang_cl_options.version = CLANG_CL_MIN_VER
+                build_tests("clang-cl", clang_cl_options)
+            end
             if not clang_options.stdmodule then
                 build_tests("llvm", clang_libcpp_options)
                 build_tests("clang", clang_libcpp_options)

xmake/rules/c++/modules/clang/builder.lua

@@ -182,7 +182,6 @@ function _get_requiresflags(target, module)
             end
         end
         requiresflags = table.unique(requiresflags)
-        -- table.sort(requiresflags)
         support.memcache():set2(cachekey, "requiresflags", requiresflags)
         support.memcache():set2(cachekey, "oldrequires", requires)
     end

xmake/rules/c++/modules/config.lua

@@ -67,6 +67,10 @@ function main(target)
             wprint("build.c++.modules.tryreuse.discriminate_on_defines is deprecated, please use build.c++.modules.reuse.strict")
         end
 
+        if target:has_tool("cxx", "clang_cl") then
+            target:set("policy", "build.c++.modules.two_phases", false)
+        end
+
         -- if containes modules, enable objectfiles output of c++.build.modules.builder
         local rule = target:rule("c++.build.modules.builder")
         rule = rule:clone()

xmake/rules/c++/modules/scanner.lua

@@ -389,7 +389,12 @@ function _patch_sourcebatch(target, sourcebatch)
 
                 local can_reuse = nocheck or _are_flags_compatible(target, dep, sourcefile, {strict = strict})
                 if can_reuse then
-                    support.set_reused(target, dep, sourcefile)
+                    local _reused, from = support.is_reused(dep, sourcefile)
+                    if _reused then
+                        support.set_reused(target, from, sourcefile)
+                    else
+                        support.set_reused(target, dep, sourcefile)
+                    end
                     table.insert(reused, sourcefile)
                     if dep:is_moduleonly() then
                         dep:data_set("cxx.modules.reused", true)
@@ -417,10 +422,15 @@ function _patch_sourcebatch(target, sourcebatch)
             if reused:has(sourcefile) then
                 local dep = target:dep(fileconfig.external)
                 assert(dep, "dep target <%s> for <%s> not found", fileconfig.external, target:fullname())
+                local _reused, from = support.is_reused(dep, sourcefile)
+                if _reused then
+                    support.set_reused(target, from, sourcefile)
+                else
+                    support.set_reused(target, dep, sourcefile)
+                end
                 if dep:is_moduleonly() then
                     dep:data_set("cxx.modules.reused", true)
                 end
-                support.set_reused(target, dep, sourcefile)
             end
             target:fileconfig_add(sourcefile, fileconfig)
         end