Looks good.<div><br></div><div>Do we want a fixme to remove this if/when *every* type has a TSI? Maybe that day will never arrive...<br><div><br><div class="gmail_quote">On Fri, Sep 3, 2010 at 4:08 PM, Zhanyong Wan (λx.x x) <span dir="ltr"><<a href="mailto:wan@google.com">wan@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Hi,<br>
<br>
I have a patch that fixes the crash mentioned in<br>
<a href="http://llvm.org/bugs/show_bug.cgi?id=8043" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=8043</a>.  Can someone please review<br>
it?<br>
<br>
You can find the patch either at<br>
<br>
<a href="http://codereview.appspot.com/2098043" target="_blank">http://codereview.appspot.com/2098043</a><br>
<br>
or at the end of this message.  Thanks!<br>
<br>
--<br>
Zhanyong<br>
<br>
Index: lib/AST/TemplateBase.cpp<br>
===================================================================<br>
--- lib/AST/TemplateBase.cpp    (revision 113025)<br>
+++ lib/AST/TemplateBase.cpp    (working copy)<br>
@@ -125,19 +125,22 @@<br>
   switch (Argument.getKind()) {<br>
   case TemplateArgument::Expression:<br>
     return getSourceExpression()->getSourceRange();<br>
-<br>
+<br>
   case TemplateArgument::Declaration:<br>
     return getSourceDeclExpression()->getSourceRange();<br>
-<br>
+<br>
   case TemplateArgument::Type:<br>
-    return getTypeSourceInfo()->getTypeLoc().getSourceRange();<br>
-<br>
+    if (TypeSourceInfo *TSI = getTypeSourceInfo())<br>
+      return TSI->getTypeLoc().getSourceRange();<br>
+    else<br>
+      return SourceRange();<br>
+<br>
   case TemplateArgument::Template:<br>
     if (getTemplateQualifierRange().isValid())<br>
       return SourceRange(getTemplateQualifierRange().getBegin(),<br>
                          getTemplateNameLoc());<br>
     return SourceRange(getTemplateNameLoc());<br>
-<br>
+<br>
   case TemplateArgument::Integral:<br>
   case TemplateArgument::Pack:<br>
   case TemplateArgument::Null:<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div></div>