[PATCH] D150002: [AMDGPU] Fix crash with 160-bit p7's by manually defining getPointerTy

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 12 02:22:02 PDT 2023


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

In D150002#4335104 <https://reviews.llvm.org/D150002#4335104>, @krzysz00 wrote:

> Ping to land this because there's active breakage? @foad @arsenm

Yes please land this.



================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:990
+MVT SITargetLowering::getPointerTy(const DataLayout &DL, unsigned AS) const {
+  if (AMDGPUAS::BUFFER_FAT_POINTER == AS && DL.getPointerSizeInBits(AS) > 128)
+    return MVT::v5i32;
----------------
Nit: why do you need the getPointerSizeInBits test?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150002



More information about the llvm-commits mailing list