Skip to content

Commit

Permalink
Bug 1726474 - Use clang.exe to build wasm files when the clang plugin…
Browse files Browse the repository at this point in the history
… is enabled. r=firefox-build-system-reviewers,andi

Differential Revision: https://phabricator.services.mozilla.com/D123591
  • Loading branch information
glandium committed Aug 25, 2021
1 parent ebf3e96 commit 2f70492
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions config/static-checking-config.mk
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,13 @@ ifdef ENABLE_CLANG_PLUGIN
# Replace "clang-cl.exe" to "clang.exe --driver-mode=cl" to avoid loading the
# module clang.exe again when load the plugin dll, which links to the import
# library of clang.exe.
# Likewise with "clang++.exe", replacing it with "clang.exe --driver-mode=g++",
# when building .wasm files from source (we do need to keep clang++ when
# building $(WASM_ARCHIVE)). Note we'd normally use $(CPPWASMOBJS), but it's
# not defined yet when this file is included.
ifeq ($(OS_ARCH),WINNT)
CC := $(subst clang-cl.exe,clang.exe --driver-mode=cl,$(CC:.EXE=.exe))
CXX := $(subst clang-cl.exe,clang.exe --driver-mode=cl,$(CXX:.EXE=.exe))
$(notdir $(addsuffix .$(WASM_OBJ_SUFFIX),$(basename $(WASM_CPPSRCS)))): WASM_CXX := $(subst clang++.exe,clang.exe --driver-mode=g++,$(WASM_CXX:.EXE=.exe))
endif
endif

0 comments on commit 2f70492

Please sign in to comment.