From 5d9c4323bb53fad3257fffa1254a56567787fd86 Mon Sep 17 00:00:00 2001
From: Torsten Ruger <torsten@villataika.fi>
Date: Thu, 17 Apr 2014 15:35:55 +0300
Subject: [PATCH] test no 3

---
 lib/asm/arm/code_generator.rb |  8 ++++----
 lib/asm/arm/instruction.rb    |  1 +
 test/test_crystal.rb          | 18 ++++++++++--------
 3 files changed, 15 insertions(+), 12 deletions(-)

diff --git a/lib/asm/arm/code_generator.rb b/lib/asm/arm/code_generator.rb
index f64e9239..885a8dfa 100644
--- a/lib/asm/arm/code_generator.rb
+++ b/lib/asm/arm/code_generator.rb
@@ -1,6 +1,6 @@
 require 'asm/arm/arm_assembler'
 require 'asm/arm/instruction'
-require 'asm/label_object'
+require_relative 'generator_label'
 require 'asm/nodes'
 require 'stream_reader'
 require 'stringio'
@@ -45,7 +45,7 @@ class Asm::Arm::CodeGenerator
         node.args << Asm::LabelRefArgNode.new { |n|
           n.label = arg.to_s
         }
-      elsif (arg.is_a?(GeneratorLabel) or arg.is_a?(GeneratorExternLabel))
+      elsif (arg.is_a?(Asm::Arm::GeneratorLabel) or arg.is_a?(Asm::Arm::GeneratorExternLabel))
         node.args << arg
       else
         raise 'Invalid argument `%s\' for instruction' % arg.inspect
@@ -75,11 +75,11 @@ class Asm::Arm::CodeGenerator
   }
 
   def label
-    GeneratorLabel.new(@asm)
+    Asm::Arm::GeneratorLabel.new(@asm)
   end
 
   def label!
-    lbl = GeneratorLabel.new(@asm)
+    lbl = Asm::Arm::GeneratorLabel.new(@asm)
     lbl.set!
     lbl
   end
diff --git a/lib/asm/arm/instruction.rb b/lib/asm/arm/instruction.rb
index cfd2eef3..2d7536c8 100644
--- a/lib/asm/arm/instruction.rb
+++ b/lib/asm/arm/instruction.rb
@@ -1,3 +1,4 @@
+require "asm/assembly_error"
 require "asm/arm/instruction_tools"
 require "asm/arm/builder_a"
 require "asm/arm/builder_b"
diff --git a/test/test_crystal.rb b/test/test_crystal.rb
index 1c70e4e3..db90d8c1 100644
--- a/test/test_crystal.rb
+++ b/test/test_crystal.rb
@@ -4,6 +4,7 @@ require "asm/arm/code_generator"
 # try  to test that the generation of basic instructions works
 # one instruction at a time, reverse testing from objdump --demangle -Sfghxp
 # tests are named as per assembler code, ie test_mov testing mov instruction
+#  adc add and bic eor orr rsb rsc sbc sub mov mvn cmn cmp teq tst b bl bx swi strb
 
 class TestArmAsm < MiniTest::Test
   # need a code generator, for arm 
@@ -14,7 +15,7 @@ class TestArmAsm < MiniTest::Test
   # code is what the generator spits out, at least one instruction worth (.first)
   # the op code is wat was witten as assembler in the first place and the binary result
   # is reversed and in 4 bytes as ruby can only do 31 bits and so we can't test with just one int (?)
-  def assert_code code , op , should
+  def assert_code code , op , should , status = false
     assert_equal op ,  code.opcode
     binary = @generator.assemble
     assert_equal 4 , binary.length
@@ -23,23 +24,24 @@ class TestArmAsm < MiniTest::Test
       assert_equal should[index] , byte 
       index += 1
     end
+    assert code.affect_status if status #no s at the end, silly for mov anyway
+  end
+  def test_adc
+    code = @generator.instance_eval { adc	r1, r3, r5}.first
+    assert_code code , :adc , [0x05,0x10,0xa3,0xe0]
   end
   def test_mov
     code = @generator.instance_eval { mov r0, 5 }.first
     assert_code code , :mov , [0x05,0x00,0xa0,0xe3]
-    assert ! code.affect_status #no s at the end, silly for mov anyway
   end
-  def test_sub
+  def test_subs
     code = @generator.instance_eval { subs r2, r0, 1 }.first
-    assert_code code, :sub ,  [0x01,0x20,0x50,0xe2]
-    assert code.affect_status #the s at the end
+    assert_code code, :sub ,  [0x01,0x20,0x50,0xe2] , true
   end
-
   def saved_other
     @generator.instance_eval do
       mov r0, 5
-      loop_start = label
-      loop_start.set!
+      loop_start = label!
       subs r0, r0, 1
       bne loop_start
       bx lr