[PATCH] D27351: [CUDA] Forward sanitizer support to host toolchain

2016-12-02 Thread Jason Henline via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL288512: [CUDA] Forward sanitizer support to host toolchain (authored by jhen). Changed prior to commit: https://reviews.llvm.org/D27351?vs=80082&id=80084#toc Repository: rL LLVM https://reviews.llvm

[PATCH] D27351: [CUDA] Forward sanitizer support to host toolchain

2016-12-02 Thread Jason Henline via Phabricator via cfe-commits
jhen created this revision. jhen added a reviewer: jlebar. jhen added subscribers: cfe-commits, hfinkel. This is an improvement on https://reviews.llvm.org/rL288448 where address sanitization was listed as supported for the CudaToolChain. Since the intent is for the CudaToolChain not to reject an

[PATCH] D27328: Remove faulty test from rL288448

2016-12-01 Thread Jason Henline via Phabricator via cfe-commits
jhen updated this revision to Diff 80012. jhen added a comment. - Switch to more specific error https://reviews.llvm.org/D27328 Files: test/Driver/cuda-no-sanitizers.cu Index: test/Driver/cuda-no-sanitizers.cu === --- test/Driv

[PATCH] D27328: Remove faulty test from rL288448

2016-12-01 Thread Jason Henline via Phabricator via cfe-commits
jhen created this revision. jhen added a reviewer: jlebar. jhen added a subscriber: cfe-commits. The test introduced by https://reviews.llvm.org/rL288448 is currently failing because unimportant but unexpected errors appear as output from a test compile line. This patch removes the faulty test fo

[PATCH] D27316: [CUDA] Filter out dirver sanitizer args for NVPTX

2016-12-01 Thread Jason Henline via Phabricator via cfe-commits
jhen updated this revision to Diff 80003. jhen added a comment. - "Support" ASAN in CudaToolChain https://reviews.llvm.org/D27316 Files: lib/Driver/ToolChains.cpp lib/Driver/ToolChains.h test/Driver/cuda-no-sanitizers.cu Index: test/Driver/cuda-no-sanitizers.cu =

[PATCH] D27316: [CUDA] Filter out dirver sanitizer args for NVPTX

2016-12-01 Thread Jason Henline via Phabricator via cfe-commits
jhen added a comment. In https://reviews.llvm.org/D27316#611162, @kcc wrote: > I am not sure this is going to work. > You essentially break on the first iteration in the beginning of the loop. > Why not exit the function before the loop? > Also, the loop "claims" the args and by breaking ea

[PATCH] D27316: [CUDA] Filter out dirver sanitizer args for NVPTX

2016-12-01 Thread Jason Henline via Phabricator via cfe-commits
jhen added a reviewer: kcc. jhen added a comment. Before this patch, the following command would fail: clang++ -fsanitize=address --cuda-gpu-arch=sm_20 -c example.cu with error: clang-4.0: error: unsupported option '-fsanitize=address' for target 'nvptx64-nvidia-cuda' After this patch, th

[PATCH] D27316: [CUDA] Filter out dirver sanitizer args for NVPTX

2016-12-01 Thread Jason Henline via Phabricator via cfe-commits
jhen created this revision. jhen added a reviewer: jlebar. jhen added a subscriber: cfe-commits. This adds to the work from https://reviews.llvm.org/rL281680. This patch takes care of another execution path in which sanitizer arguments can be considered for NVPTX because they are used for the corr