[PATCH] D85584: AMDGPU/GlobalISel: Legalize odd sized loads with widening

[Jay Foad via Phabricator via llvm-commits]

foad added a comment.

Looks OK to me but I don't understand all the details. @mbrkusanin could you take a look at this one?



================
Comment at: llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp:2303-2308
+  if (Ty.isVector()) {
+    const LLT EltTy = Ty.getElementType();
+    return LLT::vector(PowerOf2Ceil(Ty.getNumElements()), EltTy);
+  }
+
+  return LLT::scalar(PowerOf2Ceil(Ty.getSizeInBits()));
----------------
`Ty.changeElementSize(PowerOf2Ceil(Ty.getScalarSizeInBits()))`?


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

https://reviews.llvm.org/D85584