[X86] Clean up test/CodeGen/X86/2006-03-01-InstrSchedBug.ll

[jgorbe]

Summary

• Migrated from grep to FileCheck.
• Re-indented, removed boilerplate comments.
• Added 'entry' label at beginning of basic block.

[RKSimon]

test/CodeGen/X86/2006-03-01-InstrSchedBug.ll

1

Might as well regenerate the test file with the update script

3

For a trivial test like this removing the -march and replacing with this triple (-mtriple=i686-unknown-unknown) will tidy things up more.

6

I'd recommend not adding this as it might affect the reproducibility of the exact codegen that caused the old bug (unlikely but maybe).

[jgorbe]

• Regenerated the test with update_llc_test_checks.py
• Specified target triple in the RUN: line with -mtriple
• Removed entry label added in the previous version of the patch.

Thanks for the review! (and sorry it took me so long to respond)

[RKSimon]

test/CodeGen/X86/2006-03-01-InstrSchedBug.ll

3

Still no -mtriple ?

[jgorbe]

Added -mtriple parameter to the RUN line. I messed up when regenerating the test with the script, reverted it, and lost the edit I had done before. Thanks for noticing.

[RKSimon]

LGTM

test/CodeGen/X86/2006-03-01-InstrSchedBug.ll

-; RUN: llc < %s -march=x86 | not grep "subl.*%esp"
+; RUN: llc < %s -march=x86 | FileCheck %s
+
+target triple = "i686-unknown-unknown"
 
 define i32 @f(i32 %a, i32 %b) {
-        %tmp.2 = mul i32 %a, %a         ; <i32> [#uses=1]
-        %tmp.5 = shl i32 %a, 1          ; <i32> [#uses=1]
-        %tmp.6 = mul i32 %tmp.5, %b             ; <i32> [#uses=1]
-        %tmp.10 = mul i32 %b, %b                ; <i32> [#uses=1]
-        %tmp.7 = add i32 %tmp.10, %tmp.2                ; <i32> [#uses=1]
-        %tmp.11 = add i32 %tmp.7, %tmp.6                ; <i32> [#uses=1]
-        ret i32 %tmp.11
+entry:
+  %tmp.2 = mul i32 %a, %a
+  %tmp.5 = shl i32 %a, 1
+  %tmp.6 = mul i32 %tmp.5, %b
+  %tmp.10 = mul i32 %b, %b
+  %tmp.7 = add i32 %tmp.10, %tmp.2
+  %tmp.11 = add i32 %tmp.7, %tmp.6
+; CHECK-NOT: subl %esp
 }