[PATCH] D68632: [X86] Make memcmp() use PTEST if possible and also enable AVX1

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:42574
+      if (Subtarget.hasBWI()) {
+        SDValue Mask = DAG.getConstant(0xFFFFFFFFFFFFFFFF, DL, MVT::i64);
+        return DAG.getSetCC(DL, VT, DAG.getBitcast(MVT::i64, Cmp), Mask, CC);
----------------
SDValue Mask = DAG.getAllOnesConstant(DL, MVT::i64)


================
Comment at: lib/Target/X86/X86ISelLowering.cpp:42577
+      }
+      SDValue Mask = DAG.getConstant(0xFFFF, DL, MVT::i16);
+      return DAG.getSetCC(DL, VT, DAG.getBitcast(MVT::i16, Cmp), Mask, CC);
----------------
DAG.getAllOnesConstant(DL, MVT::i16)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68632/new/

https://reviews.llvm.org/D68632