<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 10, 2014 at 3:26 AM, Alexey Samsonov <span dir="ltr"><<a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="">================<br>
Comment at: lib/asan/asan_dll_thunk.cc:238<br>
@@ -236,3 +237,3 @@<br>
<br>
 INTERFACE_FUNCTION(__asan_memcpy);<br>
 INTERFACE_FUNCTION(__asan_memset);<br>
----------------<br>
</div><div class="">Timur Iskhodzhanov wrote:<br>
> The thing is that `asan_interface_internal.h` defines pretty much the same set of functions that I do here using the `INTERFACE_FUNCTION` macro.  For editing simplicity reasons, the `INTERFACE_FUNCTION` macro doesn't know the argument types of the function, so we end up with conflicts between two declarations:<br>

><br>
>   asan_dll_thunk.cc(238) : error C2733: '__asan_memcpy' : second C linkage of overloaded function not allowed<br>
>         asan_interface_internal.h(156) : see declaration of '__asan_memcpy'<br>
>   asan_dll_thunk.cc(238) : error C2440: 'type cast' : cannot convert from 'overloaded-function' to '__sanitizer::uptr'<br>
>         Context does not allow for disambiguation of overloaded function<br>
><br>
> This is basically the reason why I haven't just used `__asan_init` in the first place...<br>
> Can we move declarations/definitions between headers somehow or have a minimalistic declaration-only new header for `__asan_init`?<br>
</div>Yeah, let's create asan_init_version.h which would hold all the comments about ABI versions and have<br>
  #define ASAN_VERSIONED_INIT __asan_init_v4<br>
and get rid of all __asan_init mentions throughout the code.<br></blockquote><div><br></div><div>sounds good, do it. </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
Kostya, are you OK with that?<br>
<br>
<a href="http://reviews.llvm.org/D4411" target="_blank">http://reviews.llvm.org/D4411</a><br>
<br>
<br>
</blockquote></div><br></div></div>