Make --strip-debug configurable (for release builds)

This commit is contained in:
Fabian 2018-10-01 20:30:30 -03:00
parent 53bd41614f
commit 2ee68c960a
2 changed files with 13 additions and 2 deletions

View file

@ -14,6 +14,11 @@ JIT_DEPENDENCIES=$(GEN_DEPENDENCIES) gen/generate_jit.js
INTERPRETER_DEPENDENCIES=$(GEN_DEPENDENCIES) gen/generate_interpreter.js
ANALYZER_DEPENDENCIES=$(GEN_DEPENDENCIES) gen/generate_analyzer.js
STRIP_DEBUG=
ifeq ($(strip_debug),true)
STRIP_DEBUG=--v86-strip-debug
endif
all: build/v86_all.js build/libv86.js build/v86oxide.wasm
all-debug: build/libv86-debug.js build/v86oxide-debug.wasm
browser: build/v86_all.js
@ -83,7 +88,7 @@ CARGO_FLAGS=\
--target wasm32-unknown-unknown \
-- \
-C linker=tools/rust-lld-wrapper \
-C link-args="--import-table --global-base=8388608" \
-C link-args="--import-table --global-base=8388608 $(STRIP_DEBUG)" \
--verbose
CORE_FILES=const.js config.js io.js main.js lib.js coverage.js ide.js pci.js floppy.js \

View file

@ -10,14 +10,20 @@ from os import path
def main():
args = sys.argv[1:]
strip_debug = "--v86-strip-debug" in args
# filter out args inserted by rustc
TO_REMOVE = {
"--export-table",
"--stack-first",
"--strip-debug", # TODO: Make this configurable
"--strip-debug",
"--v86-strip-debug",
}
args = list(filter(lambda arg: arg not in TO_REMOVE, args))
if strip_debug:
args += ["--strip-debug"]
lld = find_rust_lld()
result = subprocess.run([lld] + args, stdout=subprocess.PIPE, stderr=subprocess.PIPE)