[compiler-rt] r215708 - [ASan/Win] Remove a hack that seems not to be required with VS2013 anymore

[Timur Iskhodzhanov]

Oh, vs2012... Sure, _MSC_VER SGTM.  Can you please add it?  I don't have
vs2012 around to test.
19 авг. 2014 г. 0:05 пользователь "Hans Wennborg" <hwennborg at google.com>
написал:

>
> http://llvm.org/docs/GettingStarted.html#host-c-toolchain-both-compiler-and-standard-library
>
> On Fri, Aug 15, 2014 at 6:45 AM, Timur Iskhodzhanov <timurrrr at google.com>
> wrote:
> > Author: timurrrr
> > Date: Fri Aug 15 08:45:23 2014
> > New Revision: 215708
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=215708&view=rev
> > Log:
> > [ASan/Win] Remove a hack that seems not to be required with VS2013
> anymore
> >
> > Modified:
> >     compiler-rt/trunk/lib/sanitizer_common/sanitizer_printf.cc
> >
> > Modified: compiler-rt/trunk/lib/sanitizer_common/sanitizer_printf.cc
> > URL:
> http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/sanitizer_printf.cc?rev=215708&r1=215707&r2=215708&view=diff
> >
> ==============================================================================
> > --- compiler-rt/trunk/lib/sanitizer_common/sanitizer_printf.cc (original)
> > +++ compiler-rt/trunk/lib/sanitizer_common/sanitizer_printf.cc Fri Aug
> 15 08:45:23 2014
> > @@ -22,10 +22,6 @@
> >  #include <stdio.h>
> >  #include <stdarg.h>
> >
> > -#if SANITIZER_WINDOWS && !defined(va_copy)
> > -# define va_copy(dst, src) ((dst) = (src))
> > -#endif
>
> I think VS2012 is stills supposed to be supported, and it's what's
> used to build the weekly snapshots and releases. I've put this back in
> r215930. Maybe we should guard it on _MSC_VER?
>
> Thanks,
> Hans
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140819/2584dfa9/attachment.html>