<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Mar 9, 2016 at 1:32 PM Tim Northover via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 9 March 2016 at 13:20, David Blaikie via cfe-commits<br>
<<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>> wrote:<br>
> If we are touching these tests at all, as a cleanup, why aren't we adjusting<br>
> them to not pass through the llvm optimizers/code generators at all?<br>
> Generally we are pretty clear that any test in Clang looking at llvms<br>
> generated assembly is an anachronism to be removed eventually<br>
<br>
This patch should remove all checking (and generation) of assembly.<br>
But as I said, I think banning all optimizations is going too far and<br>
leads to unreadable tests (with excess loads and stores being a<br>
particular problem).<br><br></blockquote><div><br></div><div>I still think we should avoid the optimizer, but as (at least) an intermediate step this seemed like a good compromise.</div><div><br></div><div>-eric </div></div></div>