[PATCH] D83626: [AMDGPU/MemOpsCluster] Guard new mem ops clustering heuristic logic by a flag

Mahesha S via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 17 00:07:10 PDT 2020


hsmhsm added a comment.

And, further I reverted my original commmit (cc9d69385659be32178506a38b4f2e112ed01ad4 <https://reviews.llvm.org/rGcc9d69385659be32178506a38b4f2e112ed01ad4>) which had introduced faulty heuristic since I am not finding any quick solution here. Now, none of the issues should be blocked because of this faulty patch.  I will now start from scratch again to arrive at the heauristic which hopefully purley based on number of custered-bytes without mixing-up with number of clustered instructions.

Reverted commit is: 4905536086ee47f26cd13d716eff8aa6424dfdd7 <https://reviews.llvm.org/rG4905536086ee47f26cd13d716eff8aa6424dfdd7>


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83626





More information about the llvm-commits mailing list