[llvm-commits] [llvm] r120534 - in /llvm/trunk: lib/MC/ELFObjectWriter.cpp lib/Target/ARM/ARMAsmBackend.cpp test/CodeGen/ARM/2010-11-30-reloc-movt.ll

[Jason Kim]

On Tue, Nov 30, 2010 at 7:23 PM, Rafael Espíndola
<rafael.espindola at gmail.com> wrote:
>> Simply, it didn't occur to me to test that path, as I was looking at
>> .bc -> .o generation.
>
> If the assembly parser has the features you need, you should convert
> the test to a .s -> .o test. The involved code path is a lot smaller.
> This makes the test stable to codegen changes and makes it clear what
> is being tested.

It looks like llvm-mc is not quite there yet for ARM. It barfs on
about a dozen lines in the gcc generated ARM .s file.
Oh well. I'll do the conversion to .s test when AsmParser becomes more
ARM friendly.

Thanks
-jason

>
>> Thanks!
>>
>> -jason
>
> Cheers,
> Rafael
>