Minor comment on the patch itself:<div><br></div><div><div>--- test/CodeGen/X86/iabs.ll<span class="Apple-tab-span" style="white-space:pre">    </span>(revision 157837)</div><div>+++ test/CodeGen/X86/iabs.ll<span class="Apple-tab-span" style="white-space:pre">        </span>(working copy)</div>
<div>@@ -1,13 +1,18 @@</div><div> ; RUN: llc < %s -march=x86-64 -stats  |& \</div><div>-; RUN:   grep {5 .*Number of machine instrs printed}</div><div>+; RUN:   grep {4 .*Number of machine instrs printed}</div><div>
+; RUN: llc < %s -march=x86-64 | FileCheck %s</div><div><br></div><div>Don't mix and match 'grep' and FileCheck based tests.</div><div><br></div><div>Actually, I would ask that any time you're about to touch a test case that currently uses grep, first convert it to use FileCheck, submit that, and then build your patch on top of it. We would like the test suite to slowly move away from all 'grep' based assertions.</div>
<div><br></div><div>+; rdar://10695237</div><div><br></div><div>I'd also attach rdar and bug links to the actual test code & filecheck assertions relevant for that particular issue...</div><div> </div></div>