[PATCH] D57525: Allow old toolchain on sanitizer-windows
JF Bastien via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 31 14:49:44 PST 2019
jfb added a comment.
In D57525#1379506 <https://reviews.llvm.org/D57525#1379506>, @jfb wrote:
> It looks like the latest build <http://lab.llvm.org:8011/builders/sanitizer-windows/builds/41342> didn't pick up the change:
>
> Running: cmake "-DLLVM_LIT_ARGS=-sv -j 16" -GNinja -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_PDB=ON -DLLVM_ENABLE_ASSERTIONS=ON -DLLVM_TARGETS_TO_BUILD=X86 -DCMAKE_C_COMPILER=cl -DCMAKE_CXX_COMPILER=cl "C:\\b\\slave\\sanitizer-windows\\build\\llvm.src"
>
>
> @rnk do you know what's up with that? Looks like I need to put it in the stage1 args?
Alright I moved it to stage1 in r352808 (which it seems should work for both).
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57525/new/
https://reviews.llvm.org/D57525
More information about the llvm-commits
mailing list