[llvm-branch-commits] [llvm] [NFC][AMDGPU] Create tables for async-related intrinsics (PR #225488)

Sameer Sahasrabuddhe via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Tue Sep 22 22:34:23 PDT 2026


ssahasra wrote:

This is an unfamiliar use of TableGen for me, and I haven't used TableGen itself in quite some time, so my knowledge is outdated. But there is a code smell. Any scheme that needs to repeat the name of a def in a TD file seems like bad design. There should be one place in the intrinsic def itself where it should be possible to say "this is an async intrinsic". It could be a property in the record, or it could be a class that the def derives from.

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


More information about the llvm-branch-commits mailing list