[PATCH] D79381: [X86] Fix usage of Align constructing MachineMemOperands.

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: llvm/lib/Target/X86/X86ISelLowering.cpp:8691
           unsigned NumElm = SplatBitSize / VT.getScalarSizeInBits();
-          unsigned Alignment = cast<ConstantPoolSDNode>(VCP)->getAlignment();
+          MaybeAlign Alignment(cast<ConstantPoolSDNode>(VCP)->getAlignment());
           Ld = DAG.getLoad(
----------------
Ultimately, we want to get rid of the MaybeAlign overload of getLoad(); please compute the alignment explicitly here.  I guess `DAG.getEVTAlign(MVT::getVectorVT(CVT, NumElm))`?


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

https://reviews.llvm.org/D79381