[PATCH] D52057: Emit template type and value parameter DIEs for template variables.

[Matthew Voss via Phabricator via llvm-commits]

ormris added inline comments.


================
Comment at: test/DebugInfo/X86/template.ll:20
+; CHECK: DW_TAG_variable
+; CHECK-NEXT: DW_AT_name{{.*}}"var"
 ; CHECK-NOT: NULL
----------------
dblaikie wrote:
> ormris wrote:
> > dblaikie wrote:
> > > Presumably this should actually be named 'var<int>'? Not a big deal, since this is just using whatever name is in the IR To begin with - but might make the test case less confusing.
> > In this case, the template parameters are not added to the variable name.
> Why's that?
> 
> The parameters are part of the variable name for class and function templates - seems they'd be necessary/useful to disambiguate for variable templates too.
> 
> (though I do see that GCC (8.1) also doesn't seem to put template parameters on variable templates... )
I did implement that in a previous patch (D44842), but it breaks LLDB's global symbol lookup.


Repository:
  rL LLVM

https://reviews.llvm.org/D52057