[clang-tools-extra] [clangd] [Modules] Fix to correctly handle module dependencies (PR #142828)

Justin Bogner via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 10 08:11:17 PDT 2025


================
@@ -0,0 +1,96 @@
+# A smoke test to check that a simple dependency chain for modules can work.
+#
+# FIXME: This fails on the Windows ARM64 build server. Not entirely sure why as it has been tested on
+#        an ARM64 Windows VM and appears to work there.
+# UNSUPPORTED: host=aarch64-pc-windows-msvc
+#
+# RUN: rm -fr %t
+# RUN: mkdir -p %t
+# RUN: split-file %s %t
+#
+# RUN: sed -e "s|DIR|%/t|g" %t/compile_commands.json.tmpl > %t/compile_commands.json.tmp
+# RUN: sed -e "s|CLANG_CC|%clang|g" %t/compile_commands.json.tmp > %t/compile_commands.json
+# RUN: sed -e "s|DIR|%/t|g" %t/definition.jsonrpc.tmpl > %t/definition.jsonrpc.tmp
----------------
bogner wrote:

@fleeting-xx I suspect the right medium to long term solution here would be to convert these tests to unit tests, as @ChuanqiXu9 suggested. This would avoid having to mess around with the config file using `sed` since all of that set up could be done programatically.

Thanks @zmodem for the short term fix!

https://github.com/llvm/llvm-project/pull/142828


More information about the cfe-commits mailing list