[PATCH] D52570: [X86] Disable BMI BEXTR in X86DAGToDAGISel::matchBEXTRFromAnd unless we're on compiling for a CPU with single uop BEXTR

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 29 10:07:55 PDT 2018


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

LGTM with one minor cleanup of the CHECKS in bmi.ll



================
Comment at: test/CodeGen/X86/bmi.ll:447
+; X64-BMI1-FAST-NEXT:    retq
+; X86-SLOW-BEXTR-LABEL: bextr32b_load:
+; X86-SLOW-BEXTR:       # %bb.0:
----------------
cleanup?


Repository:
  rL LLVM

https://reviews.llvm.org/D52570





More information about the llvm-commits mailing list