[PATCH] D81085: [AMDGPU/MemOpsCluster] Implement new heuristic for computing max mem ops cluster size
Mahesha S via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 10 04:20:30 PDT 2020
hsmhsm added a comment.
Hi @foad,
Ok, this is interesting. Anyway, so, we need to make sure that SIInstrInfo::getMemOperandsWithOffsetWidth() will compute width for all cases. So, for the new case, that you have added, will you going to sumbit a patch which make sure that width is computed for this new case as well?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D81085/new/
https://reviews.llvm.org/D81085
More information about the llvm-commits
mailing list