<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Apr 28, 2014 at 3:10 PM, Jingyue Wu <span dir="ltr"><<a href="mailto:jingyue@google.com" target="_blank">jingyue@google.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Did you forget to include Inputs/cuda.h in the patch? <div>


<br></div><div>A minor thing: test/SemaCUDA/<a href="http://launch_bounds.cu" target="_blank">launch_bounds.cu</a> is in the DOS format. <br></div></div><div class="gmail_extra">
<br><br><div class="gmail_quote"><div>On Mon, Apr 28, 2014 at 2:33 PM, Eli Bendersky <span dir="ltr"><<a href="mailto:eliben@google.com" target="_blank">eliben@google.com</a>></span> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">


<div>
<div dir="ltr">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).<div><br></div><div>Note that this duplicates the contents of cuda.h, though it doesn't seem too bad in this case.</div>





<div><br></div><div><br></div></div></div></blockquote></div></div></blockquote><div><br></div><div>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)</div>

<div><br></div><div>Eli</div>
<div> </div></div></div></div>