<div dir="ltr">I always felt like __FUNCTION__'s string concatenation was a nice feature that __func__ lacked.  Anyway, I don't feel that strongly.<div><br></div><div>I had to modify your change to get the flag in SANITIZER_COMMON_CFLAGS in compiler-rt/CMakeLists.txt.  See r202303.  I only tested this by building asan, since the asan test suite doesn't actually work on Windows yet.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 26, 2014 at 1:39 PM, Joerg Sonnenberger <span dir="ltr"><<a href="mailto:joerg@britannica.bec.de" target="_blank">joerg@britannica.bec.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Wed, Feb 26, 2014 at 10:30:07PM +0100, Joerg Sonnenberger wrote:<br>
> On Wed, Feb 26, 2014 at 01:06:48PM -0800, Reid Kleckner wrote:<br>
> > I'd really rather not muck around in the implementors namespace like this.<br>
> >  If the compiler is actually clang-cl (which claims to be MSVC) or MSVC<br>
> > 2013 CTP, then __func__ will be available, and who knows what will happen.<br>
><br>
> Can you try the attached patch, which uses a feature test? I am assuming<br>
> that C and C++ compiler support it both, if the former has it.<br>
<br>
</div></div>Second version, apply it to the sanitizer build command as well.<br>
<span class="HOEnZb"><font color="#888888"><br>
Joerg<br>
</font></span><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>