[llvm-branch-commits] [llvm] [Offload] Add `olGetSymbolInfo[Size]` (PR #147962)

Joseph Huber via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Fri Jul 11 06:49:52 PDT 2025


jhuber6 wrote:

> @jhuber6 Given that this interface matches the interface of other handles, any change to how it fundamentally works should probably involve updating all other getInfo queries. If we do decide to replace the Size variants, I think that should be done as a separate task that touches everything.
> 
> For now, I think it makes sense to match other handle points, and leave any refactors for device info as a separate change that touches everything.

Sure, we can get in and then rework all the get info's to just have a separate query for the size. I think that's much cleaner and keeps the number of API functions more clear. I'll accept this for now but we should definitely do that later.

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


More information about the llvm-branch-commits mailing list