[PATCH] D104730: [MCA] [AMDGPU] Adding CustomBehaviour implementation for AMDGPU.

Fri Jul 9 12:15:14 PDT 2021

holland11 added a comment.

In D104730#2866216 <https://reviews.llvm.org/D104730#2866216>, @tstellar wrote:

> I add to add DISABLE_LLVM_LINK_LLVM_DYLIB in two places, in llvm/tools/llvm-mca/lib/AMDGPU/CMakeLists.txt as you did in your comment, and also to the add_llvm_tool call in llvm/tools/llvm-mca/CMakeLists.txt.  When I did that the tests pass.

I really appreciate you testing this for me. Thank you.

@andreadb Would this be reasonable to you? If I understand correctly, this would make it so that `llvm-mca`'s executable wouldn't benefit from shared lib builds. The other alternative would be for me to ask the AMDGPU devs if we can export the specific functions that are being used here. As far as I can tell, those are our only two options.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104730