[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