<div class="gmail_quote">On Tue, May 12, 2009 at 3:32 PM, Evan Cheng <span dir="ltr"><<a href="mailto:evan.cheng@apple.com">evan.cheng@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Thanks John. But the refactoring seems a bit strange to me.<br>
<br>
First of all, the convention is for the .h name to match the .cpp one.<br>
So perhaps PEI.h should be PrologueEpilogueInserter.h?</blockquote><div><br></div><div>Of course, I'll rename it, sorry about that. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
 As for the<br>
shrink wrapper, it doesn't have to be a separate pass, but can it be a<br>
separate class? Are there too much sharing of information between the<br>
two that the data passing overhead would be too high?</blockquote><div><br></div><div>A separate class will be better, there are only two maps that are shared.</div><div>I'll put the shrink wrapping code into its own class.</div>
<div><br></div><div>Thanks for the feedback!</div><div>John</div><div><br></div><div><br></div><div><br></div></div>