<div dir="ltr"><div dir="ltr"><br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, 2 May 2019, Reid Kleckner via llvm-commits wrote:<br>
<br>
> Author: rnk<br>
> Date: Thu May 2 15:11:55 2019<br>
> New Revision: 359835<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=359835&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=359835&view=rev</a><br>
> Log:<br>
> Fix check-builtins on Windows after alias changes<br>
><br>
> Modified:<br>
> compiler-rt/trunk/lib/builtins/comparedf2.c<br>
> compiler-rt/trunk/lib/builtins/comparesf2.c<br>
> compiler-rt/trunk/test/builtins/Unit/compiler_rt_logb_test.c<br>
><br>
> Modified: compiler-rt/trunk/lib/builtins/comparedf2.c<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/builtins/comparedf2.c?rev=359835&r1=359834&r2=359835&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/builtins/comparedf2.c?rev=359835&r1=359834&r2=359835&view=diff</a><br>
> ==============================================================================<br>
> --- compiler-rt/trunk/lib/builtins/comparedf2.c (original)<br>
> +++ compiler-rt/trunk/lib/builtins/comparedf2.c Thu May 2 15:11:55 2019<br>
> @@ -140,3 +140,11 @@ AEABI_RTABI int __aeabi_dcmpun(fp_t a, f<br>
> COMPILER_RT_ALIAS(__unorddf2, __aeabi_dcmpun)<br>
> #endif<br>
> #endif<br>
> +<br>
> +#if defined(_WIN32)<br>
> +// The alias mechanism doesn't work on Windows, so emit wrapper functions.<br>
<br>
On MinGW, the same __attribute__((__alias__())) is used as for ELF, so <br>
this caused duplicate definitions of these symbols. I went ahead and <br>
pushed a fix for it.<br></blockquote><div><br></div><div>Thanks! Sorry about that, I was under the impression that we didn't emit the libgcc aliases at all on Windows, but it makes sense to emit them in a mingw environment, since we would want to provide symbols that GCC-compiled code expects to exist.</div></div></div>