[PATCH] D47346: [X86] Add a new pass to fold extension into load instructions in previous BB

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 7 02:12:47 PDT 2018


RKSimon added inline comments.


================
Comment at: lib/Target/X86/X86FoldXBBExtLoad.cpp:1
+//===-- X86FixupBWInsts.cpp - Fixup Byte or Word instructions -----------===//
+//
----------------
Fix filename and brief


================
Comment at: test/CodeGen/X86/fold-xbb-ext-load.ll:8-15
+; CHECK:        movzwl  a, %esi
+; CHECK-NEXT:   calll   v1
+; CHECK-NEXT:   jmp     .LBB0_3
+; CHECK-NEXT: .LBB0_2:
+; CHECK-NEXT:   movzwl  a+2, %esi
+; CHECK-NEXT:   calll   v2
+; CHECK-NEXT: .LBB0_3:
----------------
chandlerc wrote:
> Can you craft yoru test case to be reasonable to read with the automatically generated CHECK lines? That always makes maintenance easier *if* the result is readable... not sure if these will be.
> 
> You might need to break this apart into separate functions to make it read better.
This doesn't look like its auto-generated - can you use update_llc_test_check.py?


https://reviews.llvm.org/D47346





More information about the llvm-commits mailing list