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
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
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
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
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
=
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
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
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