[PATCH] D48606: [X86] Use bts/btr/btc for single bit set/clear/complement of a variable bit position

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 27 08:22:59 PDT 2018


craig.topper added inline comments.


================
Comment at: test/CodeGen/X86/btc_bts_btr.ll:135
+; X86:       # %bb.0:
+; X86-NEXT:    movb {{[0-9]+}}(%esp), %cl
+; X86-NEXT:    movl $1, %eax
----------------
RKSimon wrote:
> Separate issue but we should really have movzbl here
FixupBWInst only promotes byte loads in loops due to code size increase.


https://reviews.llvm.org/D48606





More information about the llvm-commits mailing list