Thanks Renato.  I'm finishing up a patch for this and will post it to llvm-commits.  But one concern, to create an ARMELFStreamer as you recommend, I had to move MCELF.h to "include/llvm/MC" and added a MCELFStreamer.h to the same directory.  That okay to do?<div>

<br></div><div>-Greg</div><div><br><br><div class="gmail_quote">On Wed, Oct 17, 2012 at 7:22 AM, Renato Golin <span dir="ltr"><<a href="mailto:rengolin@systemcall.org" target="_blank">rengolin@systemcall.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 17 October 2012 15:05, Greg Fitzgerald <<a href="mailto:garious@gmail.com">garious@gmail.com</a>> wrote:<br>


> +  virtual void EmitMappingSymbol(bool IsData);<br>
<br>
</div>I'd use an enum, or have multiple internal implementations...<br>
<br>
EmitDataMappingSymbol -> { nop on base class, on ARM, prints "$d" }<br>
EmitCodeMappingSymbol -> { nop on base class, calling either<br>
EmitThumbMappingSymbol or EmitARMMappingSymbol (private) on ARM }<br>
<div class="im"><br>
<br>
> +void MCELFStreamer::EmitMappingSymbol(bool IsData) {<br>
> +  // FIXME: The following is specific to the ARM.  This should be moved<br>
> +  //        to ARMAsmBackend.<br>
<br>
</div>Maybe MCARMELFStreamer (or whatever sounds nicer than that). ARMAsm is<br>
a big bag of code and nowadays, most of it is format agnostic, I think<br>
(asm, elf).<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
--<br>
cheers,<br>
--renato<br>
<br>
<a href="http://systemcall.org/" target="_blank">http://systemcall.org/</a><br>
</div></div></blockquote></div><br></div>