<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2014-03-03 14:29 GMT+01:00 Gautier DI FOLCO <span dir="ltr"><<a href="mailto:gautier.difolco@gmail.com" target="_blank">gautier.difolco@gmail.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">  I have tested it with a newer revision and I have kept the curly brackets.<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2882" target="_blank">http://llvm-reviews.chandlerc.com/D2882</a><br>
<br>
CHANGE SINCE LAST DIFF<br>
  <a href="http://llvm-reviews.chandlerc.com/D2882?vs=7360&id=7472#toc" target="_blank">http://llvm-reviews.chandlerc.com/D2882?vs=7360&id=7472#toc</a><br>
<div class=""><br>
Files:<br>
  lib/AST/CommentCommandTraits.cpp<br>
  lib/Lex/ModuleMap.cpp<br>
  lib/Sema/SemaDeclCXX.cpp<br>
  lib/Sema/SemaTemplate.cpp<br>
  tools/c-index-test/c-index-test.c<br>
  utils/TableGen/NeonEmitter.cpp<br>
<br>
Index: lib/AST/CommentCommandTraits.cpp<br>
===================================================================<br>
--- lib/AST/CommentCommandTraits.cpp<br>
+++ lib/AST/CommentCommandTraits.cpp<br>
@@ -62,7 +62,6 @@<br>
   else if (EditDistance < BestEditDistance) {<br>
     BestCommand.clear();<br>
     BestCommand.push_back(Command);<br>
-    BestEditDistance = EditDistance;<br>
   }<br>
 }<br>
<br>
Index: lib/Lex/ModuleMap.cpp<br>
===================================================================<br>
--- lib/Lex/ModuleMap.cpp<br>
+++ lib/Lex/ModuleMap.cpp<br>
</div>@@ -1962,7 +1962,6 @@<br>
<div class="">     // Check for the 'framework' keyword, which is not permitted here.<br>
</div><div class="">     if (Framework) {<br>
       Diags.Report(StarLoc, diag::err_mmap_inferred_framework_submodule);<br>
-      Framework = false;<br>
     }<br>
</div><div class="">   } else if (Explicit) {<br>
     Diags.Report(StarLoc, diag::err_mmap_explicit_inferred_framework);<br>
</div><div class="">Index: lib/Sema/SemaDeclCXX.cpp<br>
===================================================================<br>
--- lib/Sema/SemaDeclCXX.cpp<br>
+++ lib/Sema/SemaDeclCXX.cpp<br>
</div>@@ -11458,7 +11458,6 @@<br>
<div class="">       // The "template<>" header is extraneous.<br>
       Diag(TemplateParams->getTemplateLoc(), diag::err_template_tag_noparams)<br>
         << TypeWithKeyword::getTagTypeKindName(Kind) << Name;<br>
-      isExplicitSpecialization = true;<br>
     }<br>
   }<br>
<br>
</div><div class="">Index: lib/Sema/SemaTemplate.cpp<br>
===================================================================<br>
--- lib/Sema/SemaTemplate.cpp<br>
+++ lib/Sema/SemaTemplate.cpp<br>
@@ -5929,13 +5929,10 @@<br>
                                 SourceRange(TemplateParams->getTemplateLoc(),<br>
                                             TemplateParams->getRAngleLoc()))<br>
         << 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>
<br>
   // Check that the specialization uses the same tag kind as the<br>
Index: tools/c-index-test/c-index-test.c<br>
===================================================================<br>
--- tools/c-index-test/c-index-test.c<br>
+++ tools/c-index-test/c-index-test.c<br>
</div>@@ -1329,7 +1329,7 @@<br>
<div class=""><div class="h5">     CXCursor Parent, Root;<br>
     if (clang_getCursorKind(cursor) == CXCursor_FieldDecl ) {<br>
       const char *RootParentName;<br>
-      Root = Parent = p;<br>
+      Parent = p;<br>
       do {<br>
         Root = Parent;<br>
         RootParentName = clang_getCString(clang_getCursorSpelling(Root));<br>
Index: utils/TableGen/NeonEmitter.cpp<br>
===================================================================<br>
--- utils/TableGen/NeonEmitter.cpp<br>
+++ utils/TableGen/NeonEmitter.cpp<br>
@@ -762,7 +762,6 @@<br>
   // All pointers are void* pointers.  Change type to 'v' now.<br>
   if (pntr) {<br>
     usgn = false;<br>
-    poly = false;<br>
     type = 'v';<br>
   }<br>
   // Treat half-float ('h') types as unsigned short ('s') types.<br>
</div></div></blockquote></div><br>Gentle pIng.<br>If you will, I also can split it and do a patch per dropped dead assignment.<br><br>Regards.</div></div>