<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 19 November 2013 02:50, Bill Wendling <span dir="ltr"><<a href="mailto:isanbard@gmail.com" target="_blank">isanbard@gmail.com</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"><div dir="ltr">Adding a few people so that they can give this a looking at.</div>
</blockquote><div><br></div>Hi David, Bill,<div><br></div><div>Sorry it took so long, I'm looking at it now... Would be good to get the comments of as many people as we can, since this may be a sensitive issue to some.</div>
<div><br></div><div>Some early comments:</div><div><br></div><div>The ConstantPool object is creating as many labels as there are symbols in the constant pool, will that get merged later? </div><div><br></div><div>Overall, looks good, but there is no check if the constant pools will be accessible from the instructions that use it, since they're just dumped at the end of each section. That might work for most cases, but would be good to at least add a FIXME somewhere.</div>
<div><br></div><div>I'll look at the test cases in more detail later, but it's good that there are many of them. ;)</div><div><br></div><div>cheers,</div><div>--renato</div></div></div></div>