[interpreter] Allow mixed stacks if bytecode is preserved.

test/cctest/test-compiler.cc

Index: test/cctest/test-compiler.cc
diff --git a/test/cctest/test-compiler.cc b/test/cctest/test-compiler.cc
index 68cea9bf522d5e396e728ede7148796bb30ddfb4..c0f6034ead5b5e71f4af13120060d9ca62b6fd74 100644
--- a/test/cctest/test-compiler.cc
+++ b/test/cctest/test-compiler.cc
@@ -800,7 +800,11 @@ TEST(IgnitionBaselineOnReturn) {
       "};\n"
       "return_val = f();\n"
       "is_baseline_after_return = IsBaselineCompiled(f);\n");
-  CHECK_EQ(false, GetGlobalProperty("is_baseline_in_function")->BooleanValue());
+  // TODO(4280): Remove this entire test once --ignition-preserve-bytecode is
+  // the default and the flag is removed. This test doesn't provide benefit any
+  // longer once {InterpreterActivationsFinder} is gone.
+  bool base = FLAG_ignition_preserve_bytecode;

[rmcilroy, 2016/08/22 15:33:03]

just do: "if (FLAG_ignition_preserve_bytecode) return" up at the top?

[Michael Starzinger, 2016/08/22 15:44:23]

Done.

+  CHECK_EQ(base, GetGlobalProperty("is_baseline_in_function")->BooleanValue());
   CHECK_EQ(true, GetGlobalProperty("is_baseline_after_return")->BooleanValue());
   CHECK_EQ(1234.0, GetGlobalProperty("return_val")->Number());
 }