[clang] [llvm] [AMDGPU] Convert AMDGPUResourceUsageAnalysis pass from Module to MF pass (PR #102913)

Janek van Oirschot via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 15 06:24:47 PDT 2024


JanekvO wrote:

> Is it best to leave comments on the commit from the PR comment, i.e. currently [51f72f1](https://github.com/llvm/llvm-project/commit/51f72f115b340a092c2c9f8569911b944a4efb6d) ?
> 
> I can't seem to start a review, so it will be individual comments (I think). Is there another way to handle this?

Apologies, I think some of the additional stacked PR tooling might've been better to use rather than this manual stacked PR but I just haven't looked into any of said tooling yet (Graphite?). The PR that this one depended on has been pushed and I've rebased so it should be good to go. Thanks!

https://github.com/llvm/llvm-project/pull/102913


More information about the cfe-commits mailing list