[compiler] Delete FixedDoubleArray old case

Since the AllowHandleDereference scope doesn't happen for
kNeverSerialized (see https://crrev.com/c/v8/v8/+/2402033),
there is no need to have the extra if.

Bug: v8:7790
Change-Id: I4c9f93d2e754625e7b30aee61e2b502161bd60c6
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2404770
Commit-Queue: Georg Neis <neis@chromium.org>
Auto-Submit: Santiago Aboy Solanes <solanes@chromium.org>
Reviewed-by: Georg Neis <neis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#69843}

src/compiler/js-heap-broker.cc

@@ -3209,9 +3209,7 @@ ObjectRef FixedArrayRef::get(int i) const {
 }
 
 bool FixedDoubleArrayRef::is_the_hole(int i) const {
-  if (FLAG_turbo_direct_heap_access) {
-    return object()->is_the_hole(i);
-  } else if (data_->should_access_heap()) {
+  if (data_->should_access_heap()) {
     AllowHandleDereferenceIfNeeded allow_handle_dereference(data()->kind(),
                                                             broker()->mode());
     return object()->is_the_hole(i);
@@ -3221,9 +3219,7 @@ bool FixedDoubleArrayRef::is_the_hole(int i) const {
 }
 
 double FixedDoubleArrayRef::get_scalar(int i) const {
-  if (FLAG_turbo_direct_heap_access) {
-    return object()->get_scalar(i);
-  } else if (data_->should_access_heap()) {
+  if (data_->should_access_heap()) {
     AllowHandleDereferenceIfNeeded allow_handle_dereference(data()->kind(),
                                                             broker()->mode());
     return object()->get_scalar(i);