<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Aug 18, 2015 at 3:00 PM, Richard Smith via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</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="gmail_quote"><div><div class="h5">On Fri, Aug 14, 2015 at 11:27 AM, Hans Wennborg via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>On Tue, Aug 11, 2015 at 1:41 PM, Hans Wennborg <<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>> wrote:<br>
> On Thu, Aug 6, 2015 at 1:56 PM, David Majnemer via cfe-commits<br>
> <<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>> wrote:<br>
>> Author: majnemer<br>
>> Date: Thu Aug  6 15:56:55 2015<br>
>> New Revision: 244266<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=244266&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=244266&view=rev</a><br>
>> Log:<br>
>> [ItaniumCXXABI] Don't import RTTI data for classes with key functions<br>
>><br>
>> MinGW has some pretty strange behvaior around RTTI and<br>
>> dllimport/dllexport:<br>
>> - RTTI data is never imported<br>
>> - RTTI data is only exported if the class has no key function.<br>
>><br>
>> Modified:<br>
>>     cfe/trunk/lib/CodeGen/ItaniumCXXABI.cpp<br>
>>     cfe/trunk/test/CodeGenCXX/dllimport-rtti.cpp<br>
><br>
> I tried to merge this to 3.7 (as discussed on the commit thread for<br>
> r244488), but the test doesn't pass:<br>
><br>
> --<br>
> /usr/local/google/work/llvm-3.7/cfe.src/test/CodeGenCXX/dllimport-rtti.cpp:22:13:<br>
> error: expected string not found in input<br>
> // GNU-DAG: @_ZTV1V = available_externally dllimport<br>
>             ^<br>
> <stdin>:1:1: note: scanning from here<br>
> ; ModuleID = '/usr/local/google/work/llvm-3.7/cfe.src/test/CodeGenCXX/dllimport-rtti.cpp'<br>
> ^<br>
> <stdin>:15:1: note: possible intended match here<br>
> @_ZTV1S = available_externally dllimport unnamed_addr constant [3 x<br>
> i8*] [i8* null, i8* bitcast (i8** @_ZTI1S to i8*), i8* bitcast (void<br>
> (%struct.S*)* @_ZN1S1fEv to i8*)], align 4<br>
> ^<br>
> --<br>
><br>
> I assume there's some other commit this depends on, but I haven't been<br>
> able to figure out which. Can you take a look?<br>
<br>
</div></div>Ping?<br></blockquote><div><br></div></div></div><div>Looks like the test depends on r<span style="font-size:12.8000001907349px">243090, which shouldn't go to the branch. I'd guess we could fix the test by changing that check line to "// GNU-DAG: @_ZTV1V = external" or similar. David?</span></div></div></div></div></blockquote><div><br></div><div>external linkage works as long as it's still dllimport. I was just about to give this a shot myself.</div></div></div></div>