LGTM<br><br><div class="gmail_quote">On Tue, Mar 12, 2013 at 12:30 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">

When building for Windows, using cxa-atexit causes link errors.  We<br>
already set this flag to false for Cygywin, MinGW and hexagon targets.<br>
 This patch also sets it to false for Windows targets.<br>
<br>
Note: to reproduce:<br>
<br>
class c {<br>
public:<br>
~c() { }<br>
};<br>
<br>
C c;<br>
<br>
int main() { return 0; }<br>
<br>
On Windows when built with MSVC, this will yield:<br>
<br>
test2-906565.o : error LNK2019: unresolved external symbol ___dso_handle referen<br>
ced in function ___cxx_global_var_init<br>
test2-906565.o : error LNK2019: unresolved external symbol ___cxa_atexit referen<br>
ced in function ___cxx_global_var_init<br>
a.out : fatal error LNK1120: 2 unresolved externals<br>
<br>
This patch resolves it so that you do not have to pass<br>
-fno-use-cxa-atexit in order to properly link.<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br>Joćo Matos