<p dir="ltr">Lgtm, thanks</p>
<div class="gmail_quote">On Aug 24, 2013 10:00 PM, "David Majnemer" <<a href="mailto:david.majnemer@gmail.com">david.majnemer@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi dblaikie, echristo,<br>
<br>
Previously the backend wouldn't get to see the underlying GlobalValue<br>
that corresponds to the template argument because it would be hidden by<br>
a cast at the IR level.  Instead strip the pointer casts off of the<br>
value until we see the underlying GlobalValue.<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D1508" target="_blank">http://llvm-reviews.chandlerc.com/D1508</a><br>
<br>
Files:<br>
  lib/CodeGen/CGDebugInfo.cpp<br>
  test/CodeGenCXX/debug-info-template.cpp<br>
<br>
Index: lib/CodeGen/CGDebugInfo.cpp<br>
===================================================================<br>
--- lib/CodeGen/CGDebugInfo.cpp<br>
+++ lib/CodeGen/CGDebugInfo.cpp<br>
@@ -1258,7 +1258,8 @@<br>
             cast<MemberPointerType>(T.getTypePtr()), chars);<br>
       }<br>
       llvm::DITemplateValueParameter TVP =<br>
-          DBuilder.createTemplateValueParameter(TheCU, Name, TTy, V);<br>
+          DBuilder.createTemplateValueParameter(TheCU, Name, TTy,<br>
+                                                V->stripPointerCasts());<br>
       TemplateParams.push_back(TVP);<br>
     } break;<br>
     case TemplateArgument::NullPtr: {<br>
Index: test/CodeGenCXX/debug-info-template.cpp<br>
===================================================================<br>
--- test/CodeGenCXX/debug-info-template.cpp<br>
+++ test/CodeGenCXX/debug-info-template.cpp<br>
@@ -74,7 +74,12 @@<br>
 // CHECK: [[TGIARG1]] = {{.*}}metadata !"", metadata [[CONST_GUID_PTR:![0-9]*]], { i32, i16, i16, [8 x i8] }* @_GUID_12345678_1234_1234_1234_1234567890ab, {{.*}} ; [ DW_TAG_template_value_parameter ]<br>
 // CHECK: [[CONST_GUID_PTR]] = {{.*}}, metadata [[CONST_GUID:![0-9]*]]} ; [ DW_TAG_pointer_type ] [line 0, size 64, align 64, offset 0] [from ]<br>
 // CHECK: [[CONST_GUID]] = {{.*}}, metadata [[GUID:![0-9]*]]} ; [ DW_TAG_const_type ] [line 0, size 0, align 0, offset 0] [from _GUID]<br>
-// CHECK: [[GUID]] = {{.*}} ; [ DW_TAG_structure_type ] [_GUID] [line 100, size 0, align 0, offset 0] [decl]<br>
+// CHECK: [[GUID]] = {{.*}} ; [ DW_TAG_structure_type ] [_GUID]<br>
+<br>
+// CHECK: metadata [[PTOARGS:![0-9]*]]} ; [ DW_TAG_structure_type ] [PaddingAtEndTemplate<&PaddedObj>]<br>
+// CHECK: [[PTOARGS]] = metadata !{metadata [[PTOARG1:![0-9]*]]}<br>
+// CHECK: [[PTOARG1]] = {{.*}}metadata !"", metadata [[CONST_PADDINGATEND_PTR:![0-9]*]], { i32, i8, [3 x i8] }* @PaddedObj, {{.*}} ; [ DW_TAG_template_value_parameter ]<br>
+// CHECK: [[CONST_PADDINGATEND_PTR]] = {{.*}} ; [ DW_TAG_pointer_type ] [line 0, size 64, align 64, offset 0] [from PaddingAtEnd]<br>
 struct foo {<br>
   char pad[8]; // make the member pointer to 'e' a bit more interesting (nonzero)<br>
   int e;<br>
@@ -104,3 +109,16 @@<br>
<br>
 struct __declspec(uuid("{12345678-1234-1234-1234-1234567890ab}")) uuid;<br>
 tmpl_guid<&__uuidof(uuid)> tgi;<br>
+<br>
+struct PaddingAtEnd {<br>
+  int i;<br>
+  char c;<br>
+};<br>
+<br>
+PaddingAtEnd PaddedObj = {};<br>
+<br>
+template <const PaddingAtEnd *><br>
+struct PaddingAtEndTemplate {<br>
+};<br>
+<br>
+PaddingAtEndTemplate<&PaddedObj> PaddedTemplateObj;<br>
</blockquote></div>