[llvm-branch-commits] [DirectX] Disentangle DXIL.td's op types from LLVMType. NFC (PR #104247)
Justin Bogner via llvm-branch-commits
llvm-branch-commits at lists.llvm.org
Wed Aug 14 15:47:50 PDT 2024
bogner wrote:
> Your description and the bug you are adressing make sense. One point of confusion I see now is our mapping of intrinsics to DXIL Ops. The intrinsics still all use `LLVMType`. Is that going to be problem?
I don't see why it would. LLVMType is perfectly reasonable for describing LLVM intrinsics. DXIL ops, on the other hand, use a number of specific `dx.types.*` struct types that aren't easy to describe with LLVMType and it wouldn't make sense to extend it to do so. Also, there are far fewer types that come up with DXIL ops, so the full flexibility of LLVMType isn't needed, hence this simpler approach.
https://github.com/llvm/llvm-project/pull/104247
More information about the llvm-branch-commits
mailing list