[llvm-commits] ARM ELF disassembly with integrated-as

Tim Northover t.p.northover at gmail.com
Mon Dec 3 08:20:50 PST 2012


Hi,

I've done some work on this mapping symbols patch. I think it works
properly for both CodeGen and assembly now, with all the substantial
changes isolated to an ARMELFStreamer.

I've also gone back to the original patch's style of making
MCELFStreamer a publicly visible rather than tacking ARMELFStreamer
onto the end of MCELFStreamer.cpp. I think this is a neater solution
even in this case, and will get even more so when AArch64 lands and
needs to do something similar.

> FWIW. It'll still be a whole lots of things to change and a fair bit of fragility

The fragility is my main worry. The patch rather relies on the fact
that EmitBytes and EmitValueImpl are the only functions which will
actually emit $d data into a section. Unfortunately, there's no
obvious way to assert on that so tests are the only option.

Ok to commit?

Tim.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mapping-symbols.diff
Type: application/octet-stream
Size: 34587 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121203/667fae26/attachment.obj>


More information about the llvm-commits mailing list