diff options
author | Adrian Prantl <aprantl@apple.com> | 2013-05-16 00:41:31 +0000 |
---|---|---|
committer | Adrian Prantl <aprantl@apple.com> | 2013-05-16 00:41:31 +0000 |
commit | 48b3c7dd3df79c79a7b6a943a8ed071e3309c103 (patch) | |
tree | 0a4a4221eb48580b37807d4bec1e3cbe43d8e687 /clang/lib/CodeGen/CodeGenFunction.cpp | |
parent | 5e5ff6e170faddd31ff3e08f6d7e8ad45a2db46e (diff) | |
download | bcm5719-llvm-48b3c7dd3df79c79a7b6a943a8ed071e3309c103.tar.gz bcm5719-llvm-48b3c7dd3df79c79a7b6a943a8ed071e3309c103.zip |
Clarify comment.
llvm-svn: 181959
Diffstat (limited to 'clang/lib/CodeGen/CodeGenFunction.cpp')
-rw-r--r-- | clang/lib/CodeGen/CodeGenFunction.cpp | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/clang/lib/CodeGen/CodeGenFunction.cpp b/clang/lib/CodeGen/CodeGenFunction.cpp index 3e04ae7d153..0986ae7d6a8 100644 --- a/clang/lib/CodeGen/CodeGenFunction.cpp +++ b/clang/lib/CodeGen/CodeGenFunction.cpp @@ -192,10 +192,12 @@ void CodeGenFunction::FinishFunction(SourceLocation EndLoc) { bool OnlySimpleReturnStmts = NumSimpleReturnExprs > 0 && NumSimpleReturnExprs == NumReturnExprs; // If the function contains only a simple return statement, the - // cleanup code may become the first breakpoint in the function. To - // be safe, set the debug location for it to the location of the - // return statement. Otherwise point it to end of the function's - // lexical scope. + // location before the cleanup code becomes the last useful + // breakpoint in the function, because the simple return expression + // will be evaluated after the cleanup code. To be safe, set the + // debug location for cleanup code to the location of the return + // statement. Otherwise the cleanup code should be at the end of the + // function's lexical scope. if (CGDebugInfo *DI = getDebugInfo()) { if (OnlySimpleReturnStmts) DI->EmitLocation(Builder, LastStopPoint); |