Refactor 'dart:profiler' UserTag API

runtime/vm/intrinsifier_mips.cc

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/globals.h"  // Needed here to get TARGET_ARCH_MIPS.
 #if defined(TARGET_ARCH_MIPS)
 
 #include "vm/intrinsifier.h"
 
 #include "vm/assembler.h"
@@ skipping 1765 matching lines @@
 
 void Intrinsifier::TwoByteString_equality(Assembler* assembler) {
   StringEquality(assembler, kTwoByteStringCid);
 }
 
 // On stack: user tag (+0).
 void Intrinsifier::UserTag_makeCurrent(Assembler* assembler) {
   // T1: Isolate.
   Isolate* isolate = Isolate::Current();
   __ LoadImmediate(T1, reinterpret_cast<uword>(isolate));
+  // T2: Current user tag.
+  __ lw(T2, Address(T1, Isolate::current_tag_offset()));

[zra, 2014/05/06 15:27:24]

Same here.

[Cutch, 2014/05/06 17:00:25]

Done.

   // V0: UserTag.
   __ lw(V0, Address(SP, + 0 * kWordSize));
   // Set Isolate::current_tag_.
   __ sw(V0, Address(T1, Isolate::current_tag_offset()));
   // V0: UserTag's tag.
   __ lw(V0, FieldAddress(V0, UserTag::tag_offset()));
   // Set Isolate::user_tag_.
   __ sw(V0, Address(T1, Isolate::user_tag_offset()));
   // Set return value.
   __ Ret();
-  __ delay_slot()->LoadImmediate(V0, reinterpret_cast<int32_t>(Object::null()));

[Ivan Posva, 2014/05/06 16:43:06]

Zach, can you please make sure this is not allowed?

+  __ delay_slot()->mov(V0, T2);
+}
+
+
+void Intrinsifier::UserTag_defaultTag(Assembler* assembler) {
+  Isolate* isolate = Isolate::Current();
+  // V0: Address of default tag.
+  __ LoadImmediate(V0,
+      reinterpret_cast<uword>(isolate->object_store()) +
+                              ObjectStore::default_tag_offset());
+  __ Ret();
+  __ delay_slot()->lw(V0, Address(V0, 0));
 }
 
 
 void Intrinsifier::Profiler_getCurrentTag(Assembler* assembler) {
   // V0: Isolate.
   Isolate* isolate = Isolate::Current();
   __ LoadImmediate(V0, reinterpret_cast<uword>(isolate));
   // Set return value.
   __ Ret();
   __ delay_slot()->lw(V0, Address(V0, Isolate::current_tag_offset()));
 }
 
-
-void Intrinsifier::Profiler_clearCurrentTag(Assembler* assembler) {
-  // T1: Isolate.
-  Isolate* isolate = Isolate::Current();
-  __ LoadImmediate(T1, reinterpret_cast<uword>(isolate));
-  // Set return value to Isolate::current_tag_.
-  __ lw(V0, Address(T1, Isolate::current_tag_offset()));
-  // Clear Isolate::current_tag_.
-  const int32_t raw_null = reinterpret_cast<int32_t>(UserTag::null());
-  __ LoadImmediate(T0, raw_null);
-  __ sw(T0, Address(T1, Isolate::current_tag_offset()));
-  // Clear Isolate::user_tag_.
-  __ Ret();
-  __ delay_slot()->sw(ZR, Address(T1, Isolate::user_tag_offset()));
-}
-
 }  // namespace dart
 
 #endif  // defined TARGET_ARCH_MIPS