<div dir="auto">majnemer: opinion on the above?</div><div class="gmail_extra"><br><div class="gmail_quote">On Apr 24, 2017 2:56 PM, "Nico Weber" <<a href="mailto:thakis@chromium.org">thakis@chromium.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">"Opting into the conforming mode, /permissive-, during the series of VS 2017 update is a commitment to keeping your code base clean and to fixing non-conforming constructs we fix conformance issues in Visual C++." [...] "By contrast /permissive- offers a useful conformance mode where input C++ code is interpreted according to ISO C++ rules but also allows conforming extensions necessary to compile C++ on targets supported by Visual C++."<div><br></div><div>I guess the second quote agrees with your interpretation.</div><div><br></div><div>We already diag most of the things they already mention. The one thing we don't diag by default is Wmicrosoft-enum-forward-<wbr>reference since that's only an Extension and not an ExtWarn. We don't expose -pedantic from clang-cl, so this seemed like a somewhat natural mapping to me.</div><div><br></div><div>Should /permissive- map to -Wmicrosoft instead and turn on the parts of -Wmicrosoft that are Extensions? Should we just ignore /permissive- and possibly make some of our -Wmicrosoft Extensions ExtWarns instead?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 24, 2017 at 2:10 PM, David Majnemer <span dir="ltr"><<a href="mailto:david.majnemer@gmail.com" target="_blank">david.majnemer@gmail.com</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">-pedantic means "Issue all the warnings demanded by strict ISO C and ISO C++; reject all programs that use forbidden extensions, and some other programs that do not follow ISO C and ISO C++."<div>I believe it is more akin to -fno-ms-compatibility as it disables compatibility hacks.<br></div></div><div class="m_-6363147945289420931HOEnZb"><div class="m_-6363147945289420931h5"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 24, 2017 at 11:02 AM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.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="auto">It does sound pretty similar to me from the blog post. I think this is a decent place to start from.</div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="m_-6363147945289420931m_-3989594810322791703h5">On Apr 24, 2017 11:55 AM, "David Majnemer via Phabricator via cfe-commits" <<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>> wrote:<br type="attribution"></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="m_-6363147945289420931m_-3989594810322791703h5">majnemer requested changes to this revision.<br>
majnemer added a comment.<br>
This revision now requires changes to proceed.<br>
<br>
I don't think this is correct. GDR (of Microsoft) says the behavior is different: <a href="https://www.reddit.com/r/cpp/comments/5dh7j5/visual_c_introduces_permissive_for_conformance/da5fxjj/" rel="noreferrer" target="_blank">https://www.reddit.com/r/cpp/c<wbr>omments/5dh7j5/visual_c_introd<wbr>uces_permissive_for_conformanc<wbr>e/da5fxjj/</a><br>
<br>
<br>
<a href="https://reviews.llvm.org/D32435" rel="noreferrer" target="_blank">https://reviews.llvm.org/D3243<wbr>5</a><br>
<br>
<br>
<br></div></div><span>
______________________________<wbr>_________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-commits</a><br>
</span></blockquote></div></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</blockquote></div></div>