[Mlir-commits] [mlir] [MLIR][LLVMIR] Import unregistered intrinsics via llvm.intrinsic_call (PR #128626)
Tobias Gysi
llvmlistbot at llvm.org
Tue Feb 25 23:21:21 PST 2025
================
@@ -155,9 +161,18 @@ class LLVMImportInterface
LogicalResult convertIntrinsic(OpBuilder &builder, llvm::CallInst *inst,
LLVM::ModuleImport &moduleImport) const {
// Lookup the dialect interface for the given intrinsic.
- Dialect *dialect = intrinsicToDialect.lookup(inst->getIntrinsicID());
+ // Verify the intrinsic identifier maps to an actual intrinsic.
+ llvm::Intrinsic::ID intrinId = inst->getIntrinsicID();
+ assert(intrinId != llvm::Intrinsic::not_intrinsic);
+
+ // First lookup intrinsic across different dialects for known
+ // supported converstions, examples include arm-neon, nvm-sve, etc
----------------
gysit wrote:
```suggestion
// First lookup the intrinsic across different dialects for known
// supported conversions, examples include arm-neon, nvm-sve, etc.
```
nit: typo
https://github.com/llvm/llvm-project/pull/128626
More information about the Mlir-commits
mailing list