Remove always-on flag --experimental_inline_promise_constructor

Change-Id: Ie0bd818c629bed3011212fb7c8ab81202a462501
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1798424Reviewed-by: Peter Marshall <petermarshall@chromium.org>
Commit-Queue: Georg Neis <neis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#63678}

src/compiler/js-call-reducer.cc

@@ -5676,8 +5676,6 @@ Reduction JSCallReducer::ReducePromiseConstructor(Node* node) {
   Node* effect = NodeProperties::GetEffectInput(node);
   Node* control = NodeProperties::GetControlInput(node);
 
-  if (!FLAG_experimental_inline_promise_constructor) return NoChange();
-
   // Only handle builtins Promises, not subclasses.
   if (target != new_target) return NoChange();
 

src/flags/flag-definitions.h

@@ -617,8 +617,6 @@ DEFINE_BOOL(turbo_store_elimination, true,
 DEFINE_BOOL(trace_store_elimination, false, "trace store elimination")
 DEFINE_BOOL(turbo_rewrite_far_jumps, true,
             "rewrite far to near jumps (ia32,x64)")
-DEFINE_BOOL(experimental_inline_promise_constructor, true,
-            "inline the Promise constructor in TurboFan")
 DEFINE_BOOL(
     stress_gc_during_compilation, false,
     "simulate GC/compiler thread race related to https://crbug.com/v8/8520")

test/cctest/test-debug.cc

@@ -893,7 +893,6 @@ TEST(BreakPointInlineBoundBuiltin) {
 
 TEST(BreakPointInlinedConstructorBuiltin) {
   i::FLAG_allow_natives_syntax = true;
-  i::FLAG_experimental_inline_promise_constructor = true;
   LocalContext env;
   v8::HandleScope scope(env->GetIsolate());
 

test/mjsunit/compiler/promise-constructor.js

@@ -2,7 +2,7 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-// Flags: --allow-natives-syntax --experimental-inline-promise-constructor
+// Flags: --allow-natives-syntax
 
 // We have to patch mjsunit because normal assertion failures just throw
 // exceptions which are swallowed in a then clause.

test/unittests/compiler/js-call-reducer-unittest.cc

@@ -175,12 +175,7 @@ TEST_F(JSCallReducerTest, PromiseConstructorBasic) {
                        context, frame_state, effect, control);
 
   Reduction r = Reduce(construct);
-
-  if (FLAG_experimental_inline_promise_constructor) {
-    ASSERT_TRUE(r.Changed());
-  } else {
-    ASSERT_FALSE(r.Changed());
-  }
+  ASSERT_TRUE(r.Changed());
 }
 
 // Exactly the same as PromiseConstructorBasic which expects a reduction,