<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 16, 2015 at 3:56 PM, Justin Bogner <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Currently, LLVM_HAS_INITIALIZER_LISTS is set based on<br>
<br>
  __has_feature(cxx_generalized_initializers) && !defined(_MSC_VER)<br>
<br>
Based on the docs, we should be able to drop the _MSC_VER check now that<br>
we've moved to VS2013. Can somebody confirm that?<br>
<br></blockquote><div><br></div><div>MSDN claims that VS2013 supports braced initializer list syntax: <a href="https://msdn.microsoft.com/en-us/library/dn387583.aspx">https://msdn.microsoft.com/en-us/library/dn387583.aspx</a></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
The other problem is that GCC doesn't recognize __has_feature, so this<br>
is essentially only enabled for clang. If we're keeping this check it<br>
should really check the GCC version as well - they've had these for<br>
quite a while now.<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
</blockquote></div><br></div></div>