Use EmbedderHeapTracer instead of object grouping when trace_embedder_heap flag is set

src/fast-accessor-assembler.cc

 // Copyright 2015 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/fast-accessor-assembler.h"
 
 #include "src/base/logging.h"
 #include "src/code-stubs.h"  // For CallApiCallbackStub.
 #include "src/compiler/code-stub-assembler.h"
 #include "src/handles-inl.h"
@@ skipping 38 matching lines @@
   Node* instance_type = assembler_->WordAnd(
       assembler_->LoadObjectField(object_map,
                                   Internals::kMapInstanceTypeAndBitFieldOffset,
                                   MachineType::Uint16()),
       assembler_->IntPtrConstant(0xff));
 
   // Check whether we have a proper JSObject.
   CodeStubAssembler::Variable result(assembler_.get(),
                                      MachineRepresentation::kTagged);
   CodeStubAssembler::Label is_jsobject(assembler_.get());
+  CodeStubAssembler::Label maybe_api_object(assembler_.get());
   CodeStubAssembler::Label is_not_jsobject(assembler_.get());
   CodeStubAssembler::Label merge(assembler_.get(), &result);
   assembler_->Branch(
       assembler_->WordEqual(
           instance_type, assembler_->IntPtrConstant(Internals::kJSObjectType)),
-      &is_jsobject, &is_not_jsobject);
+      &is_jsobject, &maybe_api_object);
 
   // JSObject? Then load the internal field field_no.
   assembler_->Bind(&is_jsobject);
   Node* internal_field = assembler_->LoadObjectField(
       FromId(value), JSObject::kHeaderSize + kPointerSize * field_no,
       MachineType::Pointer());
   result.Bind(internal_field);
   assembler_->Goto(&merge);
 
+  assembler_->Bind(&maybe_api_object);
+  assembler_->Branch(
+      assembler_->WordEqual(instance_type, assembler_->IntPtrConstant(
+                                               Internals::kJSApiObjectType)),
+      &is_jsobject, &is_not_jsobject);
+
   // No JSObject? Return undefined.
   // TODO(vogelheim): Check whether this is the appropriate action, or whether
   //                  the method should take a label instead.
   assembler_->Bind(&is_not_jsobject);
   Node* fail_value = assembler_->UndefinedConstant();
   result.Bind(fail_value);
   assembler_->Goto(&merge);
 
   // Return.
   assembler_->Bind(&merge);
@@ skipping 138 matching lines @@
 void FastAccessorAssembler::Clear() {
   for (auto label : labels_) {
     delete label;
   }
   nodes_.clear();
   labels_.clear();
 }
 
 }  // namespace internal
 }  // namespace v8