<div dir="rtl"><div dir="ltr">Thanks, committed r217907.<br></div><div dir="ltr"><br></div><div dir="ltr">Here are few more of the same kind, please review.</div><div dir="ltr"><br></div><div dir="ltr">Yaron</div><div dir="ltr"><br></div></div><div class="gmail_extra"><div dir="ltr"><br><div class="gmail_quote">2014-09-16 23:11 GMT+03:00 Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 .8ex;border-left:1px #ccc solid;border-right:1px #ccc solid;padding-left:1ex;padding-right:1ex">Yep, go for it.</blockquote></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 16, 2014 at 11:54 AM, Yaron Keren <span dir="ltr"><<a href="mailto:yaron.keren@gmail.com" target="_blank">yaron.keren@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="rtl"><div dir="ltr">Hi Reid,</div><div dir="ltr"><br></div><div dir="ltr">Is this good to commit?</div><span><font color="#888888"><div dir="ltr"><br></div><div dir="ltr">Yaron</div><div dir="ltr"><br></div></font></span></div><div><div><div class="gmail_extra"><br><div class="gmail_quote"><div dir="ltr">2014-09-16 0:12 GMT+03:00 Owen Anderson <span dir="ltr"><<a href="mailto:resistor@mac.com" target="_blank">resistor@mac.com</a>></span>:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><span><br><div><blockquote type="cite"><div>On Sep 15, 2014, at 1:23 PM, Yaron Keren <<a href="mailto:yaron.keren@gmail.com" target="_blank">yaron.keren@gmail.com</a>> wrote:</div><br><div><span style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;float:none;display:inline!important">I don't know if there are other users of PassManager that keep it alive, so doFinalization() and reset() sufferred code rot with regard to reseting state. Most are OK but pieces are missing here and threre. Keeping this existing structure alive will enable reusing the passManager.</span></div></blockquote></div><br></span><div>We make use of this exact functionality, and test it internally for the use cases we care about.  Unfortunately, that does not include COFF support.</div><span><font color="#888888"><div><br></div><div>—Owen</div></font></span></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></div></div>