small function naming adjustments

lib/arm/call_instruction.rb

@@ -15,18 +15,13 @@ module Arm
   class CallInstruction < ::Register::CallInstruction
     include Arm::Constants
 
-    # arm intrucioons are pretty sensible, and always 4 bytes (thumb not supported)
-    def length
-      4
-    end
-
     def initialize(first, attributes)
       super(first , attributes)
       @attributes[:update_status] = 0
       @attributes[:condition_code] = :al if @attributes[:condition_code] == nil
     end
     
-    def assemble(io, assembler)
+    def assemble(io)
       case @attributes[:opcode]
       when :b, :call
         arg = @first

lib/arm/compare_instruction.rb

@@ -11,10 +11,6 @@ module Arm
       @rn = left
       @rd = :r0
     end
-    # arm instructions are pretty sensible, and always 4 bytes (thumb not supported)
-    def length
-      4
-    end
 
     def assemble(io, assembler)
       # don't overwrite instance variables, to make assembly repeatable

lib/arm/constants.rb

@@ -115,7 +115,7 @@ module Arm
    end
 
    # arm intrucioons are pretty sensible, and always 4 bytes (thumb not supported)
-   def length
+   def mem_length
      4
    end
 

lib/arm/logic_instruction.rb

@@ -13,11 +13,6 @@ module Arm
       @immediate = 0      
     end
 
-    # arm intrucioons are pretty sensible, and always 4 bytes (thumb not supported)
-    def length
-      4
-    end
-
     def assemble(io, assembler)
       # don't overwrite instance variables, to make assembly repeatable
       left = @left

lib/arm/move_instruction.rb

@@ -19,7 +19,8 @@ module Arm
     # but not all constants fit into the part of the instruction that is left after the instruction code,
     # so large moves have to be split into two instrucitons. we handle this here, just this instruciton looks
     # longer
-    def length
+    def mem_length
+      return 4
       is_simple ? 4 : 8
     end
 
@@ -28,7 +29,7 @@ module Arm
     def is_simple
       right = @from
       if right.is_a?(Virtual::ObjectConstant)
-        r_pos = 5 #assembler.position_for(right)
+        r_pos = right.position
         # do pc relative addressing with the difference to the instuction
         # 8 is for the funny pipeline adjustment (ie pc pointing to fetch and not execute)
         right = Virtual::IntegerConstant.new( r_pos - self.position - 8 )
@@ -45,7 +46,7 @@ module Arm
       return true
     end
 
-    def assemble(io, assembler)
+    def assemble(io)
       # don't overwrite instance variables, to make assembly repeatable
       rn = @rn
       operand = @operand
@@ -53,7 +54,7 @@ module Arm
       complex = false
       right = @from
       if right.is_a?(Virtual::ObjectConstant)
-        r_pos = assembler.position_for(right)
+        r_pos = right.position
         # do pc relative addressing with the difference to the instuction
         # 8 is for the funny pipeline adjustment (ie pc pointing to fetch and not execute)
         right = Virtual::IntegerConstant.new( r_pos - self.position - 8 )