PATCH: Moving Clang CUDA tests's input file to Inputs

Eli Bendersky eliben at google.com
Mon Apr 28 15:31:27 PDT 2014


On Mon, Apr 28, 2014 at 3:10 PM, Jingyue Wu <jingyue at google.com> wrote:

> Did you forget to include Inputs/cuda.h in the patch?
>
> A minor thing: test/SemaCUDA/launch_bounds.cu is in the DOS format.
>
>
> On Mon, Apr 28, 2014 at 2:33 PM, Eli Bendersky <eliben at google.com> wrote:
>
>>  Make the tests better behaved and isolated by having all their input
>> files in Inputs/ relative to the test itself (instead of reaching out into
>> other tests' subdirectories).
>>
>> Note that this duplicates the contents of cuda.h, though it doesn't seem
>> too bad in this case.
>>
>>
>>
r207453, thanks for the review. Fixed DOS mode and added all directories (I
think the blame here was 'svn diff' - the commit itself has everything)

Eli
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140428/e372113a/attachment.html>


More information about the cfe-commits mailing list