diff options
| author | David Blaikie <dblaikie@gmail.com> | 2019-04-19 23:02:30 +0000 |
|---|---|---|
| committer | David Blaikie <dblaikie@gmail.com> | 2019-04-19 23:02:30 +0000 |
| commit | aa3bf6ce721dc9531f52b84e93b6ad182517561b (patch) | |
| tree | c6c7ff8aa930a38c61dbbe866d5014eaa451c82e /clang/test/Analysis/crash-trace.c | |
| parent | a30920c31ffed9445ddc96203812fc812936c456 (diff) | |
| download | bcm5719-llvm-aa3bf6ce721dc9531f52b84e93b6ad182517561b.tar.gz bcm5719-llvm-aa3bf6ce721dc9531f52b84e93b6ad182517561b.zip | |
Modules: Search for a visible definition of the decl context when computing visibility of a default template parameter
The code is/was already correct for the case where a parameter is a
parameter of its enclosing lexical DeclContext (functions and classes).
But for other templates (alias and variable templates) they don't create
their own scope to be members of - in those cases, they parameter should
be considered visible if any definition of the lexical decl context is
visible.
[this should cleanup the failure on the libstdc++ modules buildbot]
[this doesn't actually fix the variable template case for a
secondary/compounding reason (its lexical decl context is incorrectly
considered to be the translation unit)]
Test covers all 4 kinds of templates with default args, including a
regression test for the still broken variable template case.
Reviewers: rsmith
Differential Revision: https://reviews.llvm.org/D60892
llvm-svn: 358795
Diffstat (limited to 'clang/test/Analysis/crash-trace.c')
0 files changed, 0 insertions, 0 deletions
