<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Apr 24, 2014 at 1:29 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class=""><br><div class="gmail_quote">On Thu, Apr 24, 2014 at 1:12 PM, Rui Ueyama <span dir="ltr"><<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="overflow:hidden">Using __imp_ symbols locally is I think not a good coding style. One<br>

should just take an address using "&" operator rather than appending<br>
__imp_ prefix. However, there are programs in the wild that depends<br>
on this link.exe's behavior, so we need this feature.</div></blockquote></div><br></div>I wonder if we should have a warning about this?</div></div></blockquote><div><br></div><div>Ideally, yes, but there's no good way to print a warning message when some symbol is actually linked. In this patch I create __imp_ symbols unconditionally for all dllexported symbols. Creating them is not visible to user -- using them should be warned.</div>

<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra">I also wonder whether the cause in the wild is source code being shared between the DLL and the users of the DLL and being written "generically".</div>


</div>
</blockquote></div><br></div><div class="gmail_extra">The program I spotted in the wild is ffmpeg. That has /include:__imp_<i>sym</i> where <i>sym</i> is a defined function in ffmpeg (/include is equivalent to --undefined). Not sure why it has such option as it is basically no-op to link.exe.</div>

<div class="gmail_extra"><br></div><div class="gmail_extra">I don't think writing code generically cannot really justify to use __imp_ symbols, as you can just use "&", so I'm still wondering what this feature is really for...</div>

</div>