<div dir="ltr">Fair enough - 'preciate your perspectives/further details.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 24, 2016 at 8:44 AM, Hal Finkel <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div style="font-family:arial,helvetica,sans-serif;font-size:10pt;color:#000000"><br><hr><blockquote style="border-left:2px solid rgb(16,16,255);margin-left:5px;padding-left:5px;color:rgb(0,0,0);font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt"><b>From: </b>"Reid Kleckner via cfe-commits" <<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>><br><b>To: </b>"David Blaikie" <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>><br><b>Cc: </b>"cfe-commits" <<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>><br><b>Sent: </b>Thursday, March 24, 2016 10:39:18 AM<br><b>Subject: </b>Re: r264205 - [CUDA] Don't define __NVCC__.<span class=""><br><br><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Mar 24, 2016 at 8:30 AM, David Blaikie via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>This seems like a different tradeoff from the one Clang made for GCC compatibility (we define all the GCC macros, but then also define others so people can detect Clang). If people were just disabling features that NVCC didn't support, that seems fairly harmless - what sort of problems/difficulties did this create?</div></div></div></div></blockquote><div><br></div><div>I think Clang is aiming for less bug-for-bug compatibility with NVCC than with GCC or Clang, so it makes less sense to pretend to be them. I also expect that there are fewer checks in the wild for __NVCC__ than for _MSC_VER and __GNUC__. If we didn't *have* to define these macros to build the world, we wouldn't, and if we can get away without defining __NVCC__, that's great.</div></div></div></div></span></blockquote>+1<br><br>I don't yet think there is a precedent for other compilers to pretend to be nvcc, and I don't think we should start one if it is not necessary. I'm sure we have some checks for __NVCC__ witin DOE applications, but I'm happy to tell those users to migrate them to __CUDACC__ as appropriate as part of transitioning to supporting Clang as a CUDA compiler.<br><br> -Hal <br><span class=""><br><blockquote style="border-left:2px solid rgb(16,16,255);margin-left:5px;padding-left:5px;color:rgb(0,0,0);font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div></div></div></div></div>
<br>_______________________________________________<br>cfe-commits mailing list<br><a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br><a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br></blockquote><br><br><br></span><span class="HOEnZb"><font color="#888888">-- <br><div><span name="x"></span>Hal Finkel<br>Assistant Computational Scientist<br>Leadership Computing Facility<br>Argonne National Laboratory<span name="x"></span><br></div></font></span></div></div></blockquote></div><br></div>