[llvm-commits] Directive parsing for AsmParser patch

[Meador Inge]

On Jun 28, 2012, at 10:21 AM, Medic, Vladimir wrote:

> Hi everyone,
> did somebody had a chance to look at the patch provided? It is not a large change so I think it won't take much time to review it.

I noticed one small issue while reviewing the code.  Upon encountering a '.code16' directive when assembling for i386-unknown-unknown the patched assembler produces:

   ../llvm-trunk/test/MC/AsmParser/directive_code32.s:5:2: error: unexpected directive .code16
           .code16
           ^
   ../llvm-trunk/test/MC/AsmParser/directive_code32.s:5:9: error: .code16 not supported yet
           .code16
                  ^
The current (unpatched) assembler produces the single error:

   ../llvm-trunk/test/MC/AsmParser/directive_code32.s:5:9: error: .code16 not supported yet
           .code16
                  ^

I am not sure if the change in ordering causes other problems or this is the only one.

--
Meador Inge
CodeSourcery / Mentor Embedded
http://www.mentor.com/embedded-software