diff options
author | Tim Shen <timshen91@gmail.com> | 2017-08-03 00:18:11 +0000 |
---|---|---|
committer | Tim Shen <timshen91@gmail.com> | 2017-08-03 00:18:11 +0000 |
commit | 1efd2229f07069dac5fc2a5da97f5056021a42f7 (patch) | |
tree | 28d56acafec5c34219c794211e7a3b7cf96301a0 /clang/lib/Frontend/CompilerInvocation.cpp | |
parent | 2c27daf7c0dd74c42f454c13c0f4d8ca15d38f31 (diff) | |
download | bcm5719-llvm-1efd2229f07069dac5fc2a5da97f5056021a42f7.tar.gz bcm5719-llvm-1efd2229f07069dac5fc2a5da97f5056021a42f7.zip |
[Sema] Add a comment on an identified bug on default arguments.
Summary:
The mis-compile is triggered by internal code, but I haven't reduced it to a small piece of code. Add a FIXME here, since a decent fix doesn't seem to be trivial.
The decent fix can be changing Decl::Init to PointerUnion<Stmt *, EvaluatedStmt *, ParamVarDecl *>, and make setUninstantiatedDefaultArg take a ParamVarDecl *, which contains the Expr * as the default argument. This way, getTemplateInstantiationArgs can take that ParamVarDecl and do the right thing.
Reviewers: rsmith
Subscribers: sanjoy, cfe-commits
Differential Revision: https://reviews.llvm.org/D36253
llvm-svn: 309908
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions