support ULP opcodes in upper case

Some open-source out there uses upper case for ULP opcodes. This change
allows using such code unmodified instead of crashing with
"Unsupported opcode or directive"

Author: wnienhaus

esp32_ulp/assemble.py

@@ -275,12 +275,12 @@ def assembler_pass(self, lines):
                         continue
                 else:
                     # machine instruction
-                    func = getattr(opcodes, 'i_' + opcode, None)
+                    func = getattr(opcodes, 'i_' + opcode.lower(), None)
                     if func is not None:
                         instruction = func(*args)
                         self.append_section(instruction.to_bytes(4, 'little'), TEXT)
                         continue
-                raise Exception('Unknown opcode or directive: %s' % opcode)
+                raise ValueError('Unknown opcode or directive: %s' % opcode)
         self.finalize_sections()
 
     def assemble(self, text):

tests/assemble.py

@@ -121,6 +121,20 @@ def test_assemble_global():
     assert exported_symbols == [(0, 'counter'), (1, 'internal'), (2, 'entry')]
 
 
+def test_assemble_uppercase_opcode():
+    a = Assembler()
+    try:
+        a.assemble("  WAIT 42")
+    except ValueError as e:
+        if str(e) != "Unknown opcode or directive: WAIT":
+            # re-raise failures we didn't expect
+            raise
+        raised = True
+    else:
+        raised = False
+    assert not raised
+
+
 def test_symbols():
     st = SymbolTable({}, {}, {})
     for entry in [
@@ -180,4 +194,5 @@ def test_symbols():
 test_assemble_bss()
 test_assemble_bss_with_value()
 test_assemble_global()
+test_assemble_uppercase_opcode()
 test_symbols()