[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