<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Apr 2, 2014 at 10:03 AM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Wed, Apr 2, 2014 at 12:28 PM, Nico Rieck <<a href="mailto:nico.rieck@gmail.com">nico.rieck@gmail.com</a>> wrote:</div>
<div class="">
> That's because this patch is Sema-only, and the dllimport function is<br>
> still getting linkonce_odr linkage instead of available_externally.<br>
> I can try to split out the relevant IRGen changes and tests and put them<br>
> up for review sooner.<br>
<br>
</div>That'd make me feel slightly more comfortable, if you can work it that way.<br>
<br>
The only other thing I'd like to see is a test case for an inline<br>
function in a class (like my example from above). I didn't see<br>
anything in our existing tests, but I may have simply overlooked it.<br>
<br>
Other than my minor nits, the patch LGTM, but I'd prefer if it was<br>
committed after the IRGen changes are in.<br></blockquote><div><br></div><div>I agree. This patch removes warnings about our ignorance of dllimport / dllexport inline functions, and we don't want to have a window where we emit incorrect IR without diagnostics.</div>
<div><br></div><div>If you want to keep things incremental, you can probably remove the "attribute ignored" warnings when you fix the IRGen for dll-visible inline functions.  Feel free to commit with that change if that works for you.</div>
<div><br></div><div>Thanks for working on this!</div></div></div></div>