From 5ad257fed02b5c3086eabadb8b482a62254321e6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Storsj=C3=B6?= Date: Sun, 2 Oct 2022 10:13:53 +0300 Subject: [PATCH] Add comments about the missing $CFGUARD_ARGS when building sanitizers --- Dockerfile.dev | 1 + build-all.sh | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/Dockerfile.dev b/Dockerfile.dev index 161b06b..2714dd1 100644 --- a/Dockerfile.dev +++ b/Dockerfile.dev @@ -132,6 +132,7 @@ RUN cd test && \ # Build sanitizers. Ubsan includes from the C++ headers, so # we need to build this after libcxx. +# $CFGUARD_ARGS is intentionally omitted here. RUN ./build-compiler-rt.sh $TOOLCHAIN_PREFIX --build-sanitizers # Sanitizers on windows only support x86. diff --git a/build-all.sh b/build-all.sh index 74cec64..4f9f0a9 100755 --- a/build-all.sh +++ b/build-all.sh @@ -89,6 +89,6 @@ fi ./build-compiler-rt.sh $PREFIX $CFGUARD_ARGS ./build-libcxx.sh $PREFIX $CFGUARD_ARGS ./build-mingw-w64-libraries.sh $PREFIX $CFGUARD_ARGS -./build-compiler-rt.sh $PREFIX --build-sanitizers +./build-compiler-rt.sh $PREFIX --build-sanitizers # CFGUARD_ARGS intentionally omitted ./build-libssp.sh $PREFIX $CFGUARD_ARGS ./build-openmp.sh $PREFIX $CFGUARD_ARGS