[PATCH] [FaultMaps] Add a parser for the __llvm__faultmaps section.
Rafael EspĂndola
rafael.espindola at gmail.com
Thu Jun 18 21:02:50 PDT 2015
>>> Right now the faultmaps section is generated only for x86_64.
>>
>> Why the template then?
>
> Because the host (the binary reading and parsing the section) could be
> big-endian? llvm-objdump built for and running on a BE architecture
> should be able to parse an __llvm_faultmaps generated on x86.
>
> That said, supporting BE is definitely not something I require, and if
> you think that makes things clearer, I'm happy to just
> static_assert(host is LE) and drop the templating.
You should support a BE host, but you don't need the templates for
that. You can just pass endian::litttle to the utility functions.
Check out lib/Object/COFF*. It is LE only, but works on a BE host.
Thanks for updating the test.
Cheers,
Rafael
More information about the llvm-commits
mailing list