<div dir="ltr">Well the thing is that in VS 2015 some of those features will start working, like constexpr.  It would be nice to have those C++ features start working automatically once the LLVM macros are changed to reflect the new compiler versions.<br></div><br><div class="gmail_quote"><div dir="ltr">On Tue, May 26, 2015 at 3:20 PM Chaoren Lin <<a href="mailto:chaorenl@google.com">chaorenl@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Yeah, the the entire thing didn't compile on Windows before. Virgile Bello tried to add these in r205769 but was reverted by emaste in r205776 since it broke FreeBSD. I put these here because I think it's nicer to have minimal changes to the cxa_demangler copy.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 26, 2015 at 3:15 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>I mean I see<br>
+#define noexcept<br>
+#define constexpr<br>
+#define snprintf _snprintf<br>
+#define alignas(X) __declspec(align(X))<br>
<br>
but I dont' see corresponding -'s from anywhere else in the diff.  So it<br>
looks like this is new code, not moved from somewhere else.  Unless I'm<br>
just missing something.  But now that I think about it, it looks like it's<br>
just because all this was previously compiled out on Windows, and now it's<br>
being compiled on Windows.<br>
<br>
Since we've inlined a copy of the code already anyway, personally I don't<br>
see anything wrong with making small changes like this just to get it to<br>
compile on all platforms.  I mean I wouldn't think it wise to change some<br>
of the actual logic of the demangling, but just adding decorators on<br>
functions to get it to compile would be ok I would think?  Others may<br>
disagree though :)<br>
<br>
<br>
</span><div><div><a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D10040&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=MEqT8U_n7oNfuDW5NRbY3ZV384ZquXIYFPWmprwUdKM&m=rynPx-qxOCpQGPrcKU5P7lV3KG668fibuPP_GfU5mT8&s=TuVti5_T9TwCcrvp6MEo0TQkJJEoGZXqFFzOzBXU4ek&e=" target="_blank">http://reviews.llvm.org/D10040</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_settings_panel_emailpreferences_&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=MEqT8U_n7oNfuDW5NRbY3ZV384ZquXIYFPWmprwUdKM&m=rynPx-qxOCpQGPrcKU5P7lV3KG668fibuPP_GfU5mT8&s=mnPEaWu76MCxOQLOtWStGdSGEugsUhrt5ZVYh0tcA6s&e=" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</div></div></blockquote></div><br></div>
</blockquote></div>