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

Tom Stellard via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 7 21:10:12 PDT 2021


tstellar added a comment.

@holland11 I think it's a different issue.

In D104730#2863589 <https://reviews.llvm.org/D104730#2863589>, @holland11 wrote:

> @tstellar Is it possible that it's the same issue seen https://reviews.llvm.org/D104401 ? In which case, I'd just need to move destructors into the cpp files rather than the .h files?

I don't think it's the same issue.  I've seen similar problems in some of the unit tests that reference non-public symbols from the backends, so I think the CMake fix is the only solution.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104730



More information about the llvm-commits mailing list