[PATCH] D94406: [AMDGPU] Fix failing assert with scratch ST mode

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 11 05:44:58 PST 2021


foad added a comment.

I think the intention was that getMemOperandsWithOffsetWidth should return false if no base operands were found (perhaps MachineScheduler could assert this?): https://reviews.llvm.org/source/llvm-github/browse/main/llvm/include/llvm/CodeGen/TargetInstrInfo.h$1304

This comment is out of date anyway: https://reviews.llvm.org/source/llvm-github/browse/main/llvm/lib/Target/AMDGPU/SIInstrInfo.cpp$390


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94406



More information about the llvm-commits mailing list