[llvm-commits] [ASan] The first version of the RTL for Windows (issue 5647052)

timurrrr at google.com timurrrr at google.com
Wed Feb 8 06:49:13 PST 2012


Reviewers: kcc1, glider,

Message:
Hi Alexander, Kostya,

Can you please review this patch?

It makes some simple C tests pass when the tests are compiled with Clang
and linked with the runtime built using MSVS cl.

It has a lot of FIXMEs but I'd like to have this code under source
control and available to someone besides me.

This patch should be a no-op change for Linux and Mac.

Thanks,
Timur.


http://codereview.appspot.com/5647052/diff/4001/lib/asan/asan_internal.h
File lib/asan/asan_internal.h (right):

http://codereview.appspot.com/5647052/diff/4001/lib/asan/asan_internal.h#newcode230
lib/asan/asan_internal.h:230: #  define ASAN_USE_EXTERNAL_SYMBOLIZER
__asan::WinSymbolize
Alternative suggestions are welcome if you believe this is too hacky.

Description:
[ASan] The first version of the RTL for Windows

Please review this at http://codereview.appspot.com/5647052/

Affected files:
   M lib/asan/asan_allocator.cc
   M lib/asan/asan_interceptors.cc
   M lib/asan/asan_internal.h
   M lib/asan/asan_rtl.cc
   A lib/asan/asan_win.cc





More information about the llvm-commits mailing list