<p dir="ltr">Oh, vs2012... Sure, _MSC_VER SGTM.  Can you please add it?  I don't have vs2012 around to test.</p>
<div class="gmail_quote">19 авг. 2014 г. 0:05 пользователь "Hans Wennborg" <<a href="mailto:hwennborg@google.com">hwennborg@google.com</a>> написал:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<a href="http://llvm.org/docs/GettingStarted.html#host-c-toolchain-both-compiler-and-standard-library" target="_blank">http://llvm.org/docs/GettingStarted.html#host-c-toolchain-both-compiler-and-standard-library</a><br>
<br>
On Fri, Aug 15, 2014 at 6:45 AM, Timur Iskhodzhanov <<a href="mailto:timurrrr@google.com">timurrrr@google.com</a>> wrote:<br>
> Author: timurrrr<br>
> Date: Fri Aug 15 08:45:23 2014<br>
> New Revision: 215708<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=215708&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=215708&view=rev</a><br>
> Log:<br>
> [ASan/Win] Remove a hack that seems not to be required with VS2013 anymore<br>
><br>
> Modified:<br>
>     compiler-rt/trunk/lib/sanitizer_common/sanitizer_printf.cc<br>
><br>
> Modified: compiler-rt/trunk/lib/sanitizer_common/sanitizer_printf.cc<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/sanitizer_printf.cc?rev=215708&r1=215707&r2=215708&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/sanitizer_printf.cc?rev=215708&r1=215707&r2=215708&view=diff</a><br>

> ==============================================================================<br>
> --- compiler-rt/trunk/lib/sanitizer_common/sanitizer_printf.cc (original)<br>
> +++ compiler-rt/trunk/lib/sanitizer_common/sanitizer_printf.cc Fri Aug 15 08:45:23 2014<br>
> @@ -22,10 +22,6 @@<br>
>  #include <stdio.h><br>
>  #include <stdarg.h><br>
><br>
> -#if SANITIZER_WINDOWS && !defined(va_copy)<br>
> -# define va_copy(dst, src) ((dst) = (src))<br>
> -#endif<br>
<br>
I think VS2012 is stills supposed to be supported, and it's what's<br>
used to build the weekly snapshots and releases. I've put this back in<br>
r215930. Maybe we should guard it on _MSC_VER?<br>
<br>
Thanks,<br>
Hans<br>
</blockquote></div>