[PATCH] D17415: [X86}: Change fixup-bw-inst.ll to test output with this optimization on and off.

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 18 16:54:45 PST 2016


qcolombet added a comment.

Hi,

Sounds good to me.
Thanks Sanjay for pointing thread out and thanks Kevin for your patient with our requests!

You have a typo in your check lines or command line depending on how you what to fix that though.

See my inline comment.

Cheers,
-Quentin


================
Comment at: fixup-bw-inst.ll:18
@@ -14,2 +17,3 @@
 ; load:
-; CHECK: movzbl
+; CHECK-BWON:  movzbl
+; CHECK-BWOFF: movb
----------------
You are actually not checking anything here.

-check-prefix BWON will check prefix that starts with BWON.
If you want CHECK-BWON, you need to use -check-prefix CHECK-BWON.


http://reviews.llvm.org/D17415





More information about the llvm-commits mailing list