[PATCH] D104149: [MCA] Adding the CustomBehaviour class to llvm-mca

Patrick Holland via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 15 10:54:31 PDT 2021


holland11 added a comment.

Okay, the AMDGPU code should be trivial now. The AMDGPUCB should be an empty implementation and the changes to the scheduling model should all have been reverted. I will submit a new patch later on once we get the flags sorted out.

I also reverted the RegisterFile changes (changed the ifs back to asserts). This should be ready to be committed if it still looks good to everyone.

I still don't have commit access, so could you do the honours again Andrea? I will request commit access after this patch and then hopefully be able to commit the small "assert -> if" changes within the RegisterFile as part of their own patch by myself.

Patrick Holland patrickeholland at gmail.com

Thanks a lot for all your help and feedback with this.


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

https://reviews.llvm.org/D104149



More information about the llvm-commits mailing list