<div dir="ltr">LGTM</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 10, 2014 at 12:34 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Dmitri noticed that in r202168 doFinalization got dropped from the<br>
DataLayoutPass.<br>
<br>
If we are going to add it back, it is probably a good idea to add<br>
doInitialization too. With that a DataLayoutPass can be reused for<br>
multiple modules.<br>
<br>
Once we have doInitialization/doFinalization, it doesn't seem<br>
necessary to pass a Module to the constructor.<br>
<br>
Overall this change seems in line with the idea of making DataLayout a<br>
required part of Module. With it the only way of having a DataLayout<br>
used is to add it to the Module.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>