[PATCH] D27316: [CUDA] Filter out dirver sanitizer args for NVPTX
Jason Henline via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Dec 1 15:29:44 PST 2016
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 early you leave the args unclaimed.
>
> I don't remember this code well enough to argue about it w/o tests. :(
Thanks for taking a look. I just wanted to make sure nobody knew of a simple solution for this problem before I dug in further. Since none of us seems too familiar with this code, I'll do more investigation to try to find the right way to fix this. Once I do that I'll post it to this review (with a test or two to make sure it's working).
https://reviews.llvm.org/D27316
More information about the cfe-commits
mailing list