[llvm] [LLVM] Change Intrinsic::ID to encode target and intrinsic index (PR #113576)
Alex Voicu via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 29 15:36:55 PDT 2024
================
@@ -275,7 +275,7 @@ enum {
/// Check the operand is a specific intrinsic ID
/// - InsnID(ULEB128) - Instruction ID
/// - OpIdx(ULEB128) - Operand index
- /// - IID(2) - Expected Intrinsic ID
----------------
AlexVlx wrote:
> AMDGPU shouldn't mix r600 and amdgcn intrinsics, but there might be a handful remaining that do that mixing. @AlexVlx also wanted to use amdgcn intrinsics in spirv
We do use them for AMDGCN flavoured SPIR-V, but I admit I was not aware and have not really internalised what this change would imply in that context. I’ll try to parse it now.
https://github.com/llvm/llvm-project/pull/113576
More information about the llvm-commits
mailing list