[turbofan] Support variable size argument popping in TF-generated functions

src/code-stubs.cc

Index: src/code-stubs.cc
diff --git a/src/code-stubs.cc b/src/code-stubs.cc
index d179a472ecee71efcaacba4f758a737bbac6b1ca..9daf1d4e990e60d737cda0b1d597d35c31c4d9c6 100644
--- a/src/code-stubs.cc
+++ b/src/code-stubs.cc
@@ -2900,7 +2900,9 @@ compiler::Node* FastCloneShallowArrayStub::Generate(
       {
         Node* abort_id = assembler->SmiConstant(
             Smi::FromInt(BailoutReason::kExpectedFixedDoubleArrayMap));
-        assembler->TailCallRuntime(Runtime::kAbort, context, abort_id);
+        assembler->CallRuntime(Runtime::kAbort, context, abort_id);
+        result.Bind(assembler->UndefinedConstant());

[danno, 2016/10/25 06:43:32]

It's not a good idea to tail call with a different return signature, which can
happen easily by tail-calling abort from a stub that doesn't return a value.
This started triggering CHECKs when tinkering with this CL.

+        assembler->Goto(&return_result);
       }
       assembler->Bind(&correct_elements_map);
     }