diff options
| author | David Majnemer <david.majnemer@gmail.com> | 2015-02-26 00:57:33 +0000 |
|---|---|---|
| committer | David Majnemer <david.majnemer@gmail.com> | 2015-02-26 00:57:33 +0000 |
| commit | 5123664431100b019a989248e54d4c89b2cd05f9 (patch) | |
| tree | 5dc1627021535c1a3bc66f94cf6f84ef2026988e /clang/lib/Sema/SemaChecking.cpp | |
| parent | e2008ae475a06972854fc0fee233b0a0495b1e77 (diff) | |
| download | bcm5719-llvm-5123664431100b019a989248e54d4c89b2cd05f9.tar.gz bcm5719-llvm-5123664431100b019a989248e54d4c89b2cd05f9.zip | |
Sema: __assume with side effects shouldn't result in invalid AST nodes
We'd diagnose an __assume expression which contained a function call.
This would result in us wrongly returning ExprError, causing mysterious
failures later on.
llvm-svn: 230597
Diffstat (limited to 'clang/lib/Sema/SemaChecking.cpp')
| -rw-r--r-- | clang/lib/Sema/SemaChecking.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang/lib/Sema/SemaChecking.cpp b/clang/lib/Sema/SemaChecking.cpp index bdaba9a7ce9..eb6bffcd48e 100644 --- a/clang/lib/Sema/SemaChecking.cpp +++ b/clang/lib/Sema/SemaChecking.cpp @@ -2378,7 +2378,7 @@ bool Sema::SemaBuiltinAssume(CallExpr *TheCall) { if (Arg->isInstantiationDependent()) return false; if (Arg->HasSideEffects(Context)) - return Diag(Arg->getLocStart(), diag::warn_assume_side_effects) + Diag(Arg->getLocStart(), diag::warn_assume_side_effects) << Arg->getSourceRange() << cast<FunctionDecl>(TheCall->getCalleeDecl())->getIdentifier(); |
