[llvm-commits] Proof of concept patch for unifying the .s/ELF emission of .ARM.attributes

[Jason Kim]

On Wed, Oct 13, 2010 at 9:04 AM, Rafael Espindola <espindola at google.com> wrote:
>> Oops, I misread Rafael's sentence. Sorry about that Rafael., Didn't
>> mean to sound flamey.
>> In any case, --hex preserves old behavior. It seemed like the least
>> intrusive way to get hex dump to work.
>
> So, the constants that you want to check for are section types and
> contents, right? The contents are already dumped as hex. Can you just
> unconditionally change the section types to hex and avoid the option?

There's about 30 tests that currently depend upon the decimal output.
And maybe lots more in the test-suite.
I don't want to do that in the same patch. I'll add a simplified
--hex, and --dec (default) option switch the default once I clean up
the tests.

>
> Cheers,
> --
> Rafael Ávila de Espíndola
>