<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">You are right. Now I realize that I was a little bit too "aggressive" here. I will be more careful in the future.<div><br></div><div>- Kuba</div><div><br><div><div>On Dec 21, 2011, at 10:19 PM, Chandler Carruth wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div class="gmail_quote">On Wed, Dec 21, 2011 at 12:18 PM, Jakub Staszak <span dir="ltr"><<a href="mailto:kubastaszak@gmail.com">kubastaszak@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 id=":24p">- Change a few operator[] to lookup which is cheaper.<br></div></blockquote><div><br></div><div>Cool, but...</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div id=":24p">
- Add some constantness.</div></blockquote></div><br><div>I really don't like this. There are a lot of methods on MachineBasicBlock that are non-const, and I don't want to have to undo this patch when I discover I need to call one. I don't think this is protecting us from any bugs or enabling any interesting optimizations.</div>
</blockquote></div><br></div></body></html>