[turbofan] Properly call %TraceExit on explicit return.

src/compiler/ast-graph-builder.cc

Index: src/compiler/ast-graph-builder.cc
diff --git a/src/compiler/ast-graph-builder.cc b/src/compiler/ast-graph-builder.cc
index b236775154bf59a1962cfcfea71a4a8e9f48a0f9..5e88c86b6a7589a725094828e682d134c88d888b 100644
--- a/src/compiler/ast-graph-builder.cc
+++ b/src/compiler/ast-graph-builder.cc
@@ -1203,6 +1203,9 @@ void AstGraphBuilder::VisitBreakStatement(BreakStatement* stmt) {
 void AstGraphBuilder::VisitReturnStatement(ReturnStatement* stmt) {
   VisitForValue(stmt->expression());
   Node* result = environment()->Pop();
+  if (FLAG_trace) {
+    NewNode(javascript()->CallRuntime(Runtime::kTraceExit), result);

[Michael Starzinger, 2016/02/01 18:33:16]

This needs to go into AstGraphBuilder::BuildReturn, otherwise it is being
executed before the finally block.

[Benedikt Meurer, 2016/02/02 05:01:41]

Done.

+  }
   execution_control()->ReturnValue(result);
 }