<div dir="ltr">I'll leave it on the back burner. At least there's a fairly easy workaround for the time being.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On 9 January 2013 03:17, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Tue, Jan 8, 2013 at 4:11 PM, Will Wilson <<a href="mailto:will@indefiant.com">will@indefiant.com</a>> wrote:<br>

><br>
</div><div><div class="h5">>> > 1. I assume the same is true for the whole if-else block in<br>
>> > CheckMemberSpecialization() that applies TSK_ExplicitSpecialization for<br>
>> > various Decl types?<br>
>><br>
>> Yes, those calls to setLocation all look wrong. Do you see any test<br>
>> failures if you remove them?<br>
><br>
><br>
> Failing Tests (5):<br>
>     Clang :: CXX/temp/temp.spec/temp.expl.spec/p2-0x.cpp<br>
>     Clang :: CXX/temp/temp.spec/temp.expl.spec/p2.cpp<br>
>     Clang :: CXX/temp/temp.spec/temp.expl.spec/p5.cpp<br>
>     Clang :: CXX/temp/temp.spec/temp.explicit/p4.cpp<br>
>     Clang :: SemaCXX/warn-unused-filescoped.cpp<br>
><br>
> It looks like a few diagnostics rely on the location data. All of the fails<br>
> seem to be composed of the diagnostics moving from the specialization to the<br>
> declaration (from test/CXX/temp/temp.spec/temp.expl.spec/p5.cpp):<br>
><br>
> error: 'note' diagnostics expected but not seen:<br>
>   Line 31: forward declaration<br>
> error: 'note' diagnostics seen but not expected:<br>
>   Line 15: forward declaration of 'X<IntHolder, long>::Inner'<br>
> 2 errors generated.<br>
><br>
> Obviously the tests could be fairly easily updated but it seems there's more<br>
> than a little danger of adversely affecting utility of the diagnostics. I<br>
> guess the diagnostics would have to be reworked to ensure they resolved the<br>
> explicitly specialized definition rather than declaration location - which<br>
> sounds like it might be a fair bit of work...<br>
<br>
</div></div>OK, that diagnostic change in particular looks bad. Seems this will<br>
not be straightforward to fix. :(<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div><span style="background-color:rgb(255,255,255);color:rgb(68,68,68);font-family:Arial,Helvetica,sans-serif"><b>Indefiant Ltd.</b></span></div><div><font color="#444444" face="Arial, Helvetica, sans-serif"><span style="font-size:12px"><b><br>
</b></span></font></div><font color="#444444" face="Arial, Helvetica, sans-serif"><span style="font-size:12px">Firsby Lodge, New Main Road, Scamblesby, Louth, Lincs LN11 9XH UK<br></span></font><div style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px;padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:0px;background-color:rgb(255,255,255)">
<span style="color:rgb(68,68,68);font-family:Arial,Helvetica,sans-serif;font-size:x-small"><i>Tel: +44 20 8123 7663 England Registered No. 07936820 VAT No. </i></span><span style="background-color:transparent"><font color="#444444" face="Arial, Helvetica, sans-serif" size="1"><i>128556202</i></font></span></div>

</div>