<div dir="ltr">Seems consistent with what MSVC does.<div><br></div><div>I'd change the wording of the warning to match our other extension warnings.  Something like: "... (is|are|as) a Microsoft extension"</div>
<div><br></div><div>Otherwise, looks good.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, May 7, 2014 at 9:58 AM, Will Wilson <span dir="ltr"><<a href="mailto:will@indefiant.com" target="_blank">will@indefiant.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">Now with added patch...</div><div class="gmail_extra"><div><div class="h5"><br><br><div class="gmail_quote">
On 7 May 2014 18:57, Will Wilson <span dir="ltr"><<a href="mailto:will@indefiant.com" target="_blank">will@indefiant.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">Hi All,
<div><br></div><div>Here's a small patch to allow duplicate explicit class instantiations through with only a warning in MSVCCompat mode. MSVC itself silently ignores duplicate explicit instantiations. Clang appears to skip any duplicates so it shouldn't be too much of an issue to reduce the error to a warning for MSVC compatibility in this case.</div>


<div><br></div><div>Test case included and clang format applied. Let me know if it LGTY.</div><div><br></div><div>Cheers,</div><div>Will.</div></div>
</blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><div dir="ltr"><div><span style="background-color:rgb(255,255,255);color:rgb(68,68,68);font-family:Arial,Helvetica,sans-serif"><b>Indefiant </b>: <a href="http://www.indefiant.com" target="_blank">http://www.indefiant.com</a></span></div>

<div><span style="font-family:Arial,Helvetica,sans-serif"><font color="#999999">Home of Recode : Runtime C++ Editing for VS</font></span></div></div>
</font></span></div>
</blockquote></div><br></div>