<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2014-03-06 23:07 GMT+01:00 Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
================<br>
Comment at: lib/AST/CommentCommandTraits.cpp:65<br>
@@ -64,3 +64,2 @@<br>
<div class="">     BestCommand.push_back(Command);<br>
-    BestEditDistance = EditDistance;<br>
   }<br>
</div>----------------<br>
Fixed in r203153.<br>
<br>
================<br>
Comment at: lib/Lex/ModuleMap.cpp:1965<br>
@@ -1964,3 +1964,2 @@<br>
<div class="">       Diags.Report(StarLoc, diag::err_mmap_inferred_framework_submodule);<br>
-      Framework = false;<br>
     }<br>
</div>----------------<br>
We don't seem to use the `Framework` flag in this function at all; this looks like a bug.<br>
<br>
================<br>
Comment at: lib/Sema/SemaDeclCXX.cpp:11461<br>
@@ -11460,3 +11460,2 @@<br>
<div class="">         << TypeWithKeyword::getTagTypeKindName(Kind) << Name;<br>
-      isExplicitSpecialization = true;<br>
     }<br>
</div>----------------<br>
This one looks fine to drop, but the code is arguably more robust to future changes with this dead store.<br>
<br>
================<br>
Comment at: lib/Sema/SemaTemplate.cpp:5932-5938<br>
@@ -5931,9 +5931,6 @@<br>
<div class="">         << SourceRange(LAngleLoc, RAngleLoc);<br>
-    else<br>
-      isExplicitSpecialization = true;<br>
   } else if (TUK != TUK_Friend) {<br>
     Diag(KWLoc, diag::err_template_spec_needs_header)<br>
       << FixItHint::CreateInsertion(KWLoc, "template<> ");<br>
     TemplateKWLoc = KWLoc;<br>
-    isExplicitSpecialization = true;<br>
   }<br>
</div>----------------<br>
It seems suspicious that `isExplicitSpecialization` is unused from this point onwards, but this needs more checking.<br>
<br>
================<br>
Comment at: tools/c-index-test/c-index-test.c:1332<br>
@@ -1331,3 +1331,3 @@<br>
<div class="">       const char *RootParentName;<br>
-      Root = Parent = p;<br>
</div>       do {<br>
----------------<br>
Obviously redundant, removed in r203169.<br>
<br>
================<br>
Comment at: utils/TableGen/NeonEmitter.cpp:765<br>
@@ -764,3 +764,2 @@<br>
<div class="">     usgn = false;<br>
-    poly = false;<br>
     type = 'v';<br>
</div>----------------<br>
I don't know if `poly` is supposed to be unused from this point onwards, but even if so, this dead store could make the code robust against future changes that add later uses of `poly`.<br>
<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2882" target="_blank">http://llvm-reviews.chandlerc.com/D2882</a><br>
</blockquote></div><br></div><div class="gmail_extra">Thank you for the feedback and for your time.<br></div><div class="gmail_extra">Sorry for that.<br></div></div>