Cleaned up calling-related methods in the ARM assembler.

src/arm/assembler-arm.h

Index: src/arm/assembler-arm.h
===================================================================
--- src/arm/assembler-arm.h	(revision 8475)
+++ src/arm/assembler-arm.h	(working copy)
@@ -378,7 +378,6 @@
   INLINE(explicit Operand(int32_t immediate,
          RelocInfo::Mode rmode = RelocInfo::NONE));
   INLINE(explicit Operand(const ExternalReference& f));
-  INLINE(explicit Operand(const char* s));
   explicit Operand(Handle<Object> handle);
   INLINE(explicit Operand(Smi* value));
 
@@ -1166,6 +1165,9 @@
 
   // Debugging
 
+  // Check the code size generated from label to here.
+  int SizeOfCodeGeneratedSince(Label* l) { return pc_offset() - l->pos(); }

[Kevin Millikin (Chromium), 2011/06/30 08:55:02]

I'm not a fan of single character identifiers.  'l' is particularly bad.

[Sven Panne, 2011/06/30 09:29:48]

Me neither, but I'll have to admit being guilty of copy-n-paste. ;-) I'll change
this and the corresponding identifiers on the other platforms.

[Rodolph Perfetta, 2011/06/30 09:46:07]

There is already a method 'InstructionsGeneratedSince' in this file, do we need
both?

[Sven Panne, 2011/06/30 11:26:03]

InstructionsGeneratedSince can at least use SizeOfCodeGeneratedSince. It is a
bit unfortunate that the different assemblers don't agree in their
signatures/names for similar methods...

+
   // Mark address of the ExitJSFrame code.
   void RecordJSReturn();