<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 13 January 2014 18:34, David Peixotto <span dir="ltr"><<a href="mailto:dpeixott@codeaurora.org" target="_blank">dpeixott@codeaurora.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Ping. Any feedback on the patch/approach? What do people think about storing<br>


the constant pools in the ARMTargetStreamer class?<br></blockquote><div></div></div><br></div><div class="gmail_extra">Hi David,</div><div class="gmail_extra"><br></div><div class="gmail_extra">Sorry for the delay.</div>

<div class="gmail_extra"><br></div><div class="gmail_extra">I think it's a good idea to have an object handling the constant pools, that would enable us to merge multiple pools, de-duplicate, etc.</div><div class="gmail_extra">

<br></div><div class="gmail_extra">But the new ARMTargetMachOStreamer class being all but the constant pool is a bit too much, I think. The fact that the ASM streamer and the object streamer are separated is a mess, but we shouldn't add more to it.</div>

<div class="gmail_extra"><br></div><div class="gmail_extra">I have to say I lost touch with how the streamers are being used in the grand scheme of things, but I believe only the asm parser is now using that streamer and should move to the one the back-end is actually using. Rafael should know more...</div>
<div class="gmail_extra"><br></div><div class="gmail_extra"><br></div><div class="gmail_extra">cheers,</div><div class="gmail_extra">--renato</div><div class="gmail_extra">
<br></div></div>