Fair. Did you want to handle this a different way? I thought the list at least seemed reasonable for spelling. <br><br><div class="gmail_quote"><div dir="ltr">On Thu, Feb 8, 2018, 11:44 AM Artem Belevich via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tra added inline comments.<br>
<br>
<br>
================<br>
Comment at: test/Misc/target-invalid-cpu-note.c:38<br>
+// NVPTX: note: valid target CPU values are: sm_20, sm_21, sm_30, sm_32, sm_35,<br>
+// NVPTX-SAME: sm_37, sm_50, sm_52, sm_53, sm_60, sm_61, sm_62, sm_70, sm_72<br>
----------------<br>
Nit: Generally speaking this note is false. For any given version of CUDA, some of the listed GPU variants will not be accepted. E.g. CUDA versions before 9.1 do not know anything about sm_72, but CUDA-9.1  does not supports sm_20.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D43045" rel="noreferrer" target="_blank">https://reviews.llvm.org/D43045</a><br>
<br>
<br>
<br>
</blockquote></div>