<div class="gmail_quote">On Tue, Dec 13, 2011 at 10:53 PM, Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com">clattner@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
On Dec 13, 2011, at 3:08 PM, Matt Beaumont-Gay wrote:<br>
<br>
> Thanks for the heads up. I'm working on optimizing that codepath some more.<br>
<br>
</div>In the meantime, would it be best to temporarily revert the patch so that other performance regressions aren't lost in this noise?<br></blockquote><div><br></div><div>The test case is an extreme one. In the preprocessor it expands to an amazing number of array dereferences, many of them in trivially dead code. We end up producing warnings for a large number of them and then filter them out. Anyways, it seems very unlikely that much other code will tickle this performance problem. Regressions elsewhere should still show up.</div>
<div><br></div><div>That said, I think Matt almost had the performance fixes done, so I'm not too worried about it?</div></div>