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

Wed Jul 21 11:29:02 PDT 2021

Hey Patrick,

tl;dr: I prefer point 2.

I have replied to that thread on discourse.

-Andrea

On Wed, Jul 21, 2021 at 12:49 AM Patrick Holland via Phabricator <
reviews at reviews.llvm.org> wrote:

> holland11 added a comment.
>
> @tstellar @andreadb I've been thinking about the problem for a bit now and
> have been brainstorming different ideas and digging through the code. I
> just made a post on the llvm discourse to hopefully get some feedback on
> the issue (and some potential solutions that I've come up with). If you
> guys are curious or would be willing to take a look, here's the post
> https://llvm.discourse.group/t/need-help-deciding-how-to-access-backend-functions-from-within-a-tool/3915
> .
>
>
> Repository:
>   rG LLVM Github Monorepo
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D104730/new/
>
> https://reviews.llvm.org/D104730
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210721/90067491/attachment.html>