[PATCH] D57218: [X86] Combine masked store and truncate into masked truncating stores.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 25 00:05:55 PST 2019


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

LGTM with one minor



================
Comment at: lib/Target/X86/X86ISelLowering.cpp:37669
     if (Mask.getScalarValueSizeInBits() != 1) {
       const TargetLowering &TLI = DAG.getTargetLoweringInfo();
       APInt DemandedMask(APInt::getSignMask(VT.getScalarSizeInBits()));
----------------
remove this - its a shadow duplicate now


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57218





More information about the llvm-commits mailing list