[llvm] [KernelInfo] Implement new LLVM IR pass for GPU code analysis (PR #102944)

Johannes Doerfert via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 12 12:13:50 PDT 2024


jdoerfert wrote:

> An alternative is to have a base class with common info and then have target dependent part in sub classes.

I don't see how that buys us much and I feel there is a cost to it (more files, more places, more boilerplate). What do we gain by splitting it up? In the existing code we can/should just print target dependent features (like waves-per-eu) only for AMDGPU and call it a day, no?

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


More information about the llvm-commits mailing list