diff options
| author | Ilya Biryukov <ibiryukov@google.com> | 2017-07-21 09:24:00 +0000 |
|---|---|---|
| committer | Ilya Biryukov <ibiryukov@google.com> | 2017-07-21 09:24:00 +0000 |
| commit | b6d1ec80736788015dbb12b48d9cf6bc4c534fda (patch) | |
| tree | 882ec98797ca7dd20288f816d0ca549211e564d3 /clang/lib/Sema/SemaCodeComplete.cpp | |
| parent | ba3bd7ce3e4976983a2d04f84bb6b8ea1ed32a85 (diff) | |
| download | bcm5719-llvm-b6d1ec80736788015dbb12b48d9cf6bc4c534fda.tar.gz bcm5719-llvm-b6d1ec80736788015dbb12b48d9cf6bc4c534fda.zip | |
Fixed failing assert in code completion.
Summary:
The code was accessing uninstantiated default argument.
This resulted in failing assertion at ParmVarDecl::getDefaultArg().
Reviewers: erikjv, klimek, bkramer, krasimir
Reviewed By: krasimir
Subscribers: cfe-commits
Differential Revision: https://reviews.llvm.org/D35682
llvm-svn: 308722
Diffstat (limited to 'clang/lib/Sema/SemaCodeComplete.cpp')
| -rw-r--r-- | clang/lib/Sema/SemaCodeComplete.cpp | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/clang/lib/Sema/SemaCodeComplete.cpp b/clang/lib/Sema/SemaCodeComplete.cpp index 91a8c619b26..4de7d422072 100644 --- a/clang/lib/Sema/SemaCodeComplete.cpp +++ b/clang/lib/Sema/SemaCodeComplete.cpp @@ -2401,10 +2401,7 @@ formatBlockPlaceholder(const PrintingPolicy &Policy, const NamedDecl *BlockDecl, static std::string GetDefaultValueString(const ParmVarDecl *Param, const SourceManager &SM, const LangOptions &LangOpts) { - const Expr *defaultArg = Param->getDefaultArg(); - if (!defaultArg) - return ""; - const SourceRange SrcRange = defaultArg->getSourceRange(); + const SourceRange SrcRange = Param->getDefaultArgRange(); CharSourceRange CharSrcRange = CharSourceRange::getTokenRange(SrcRange); bool Invalid = CharSrcRange.isInvalid(); if (Invalid) |
