diff options
author | David Blaikie <dblaikie@gmail.com> | 2014-10-16 04:21:25 +0000 |
---|---|---|
committer | David Blaikie <dblaikie@gmail.com> | 2014-10-16 04:21:25 +0000 |
commit | 3b32ee4d36a1dff5fbc1a79798340c5d175ce583 (patch) | |
tree | 59d435ed5ad573fbd908eb0dda1e6f868839155e /test/Driver/clang-g-opts.c | |
parent | 87f89f4b3b6cbc70f4236654fe05770f87bb6a4f (diff) | |
download | clang-3b32ee4d36a1dff5fbc1a79798340c5d175ce583.tar.gz |
PR21246: DebugInfo: Emit the appropriate type (cv qualifiers, reference-ness, etc) for non-type template parameters
Plumb through the full QualType of the TemplateArgument::Declaration, as
it's insufficient to only know whether the type is a reference or
pointer (that was necessary for mangling, but insufficient for debug
info). This shouldn't increase the size of TemplateArgument as
TemplateArgument::Integer is still longer by another 32 bits.
Several bits of code were testing that the reference-ness of the
parameters matched, but this seemed to be insufficient (various other
features of the type could've mismatched and wouldn't've been caught)
and unnecessary, at least insofar as removing those tests didn't cause
anything to fail.
(Richard - perchaps you can hypothesize why any of these checks might
need to test reference-ness of the parameters (& explain why
reference-ness is part of the mangling - I would've figured that for the
reference-ness to be different, a prior template argument would have to
be different). I'd be happy to add them in/beef them up and add test
cases if there's a reason for them)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@219900 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test/Driver/clang-g-opts.c')
0 files changed, 0 insertions, 0 deletions