This commit changes intrinsics that have immarg parameter attributes to
model these parameters as attributes, instead of operands. Using
operands only works if the operation is an llvm.mlir.constant,
otherwise the exported LLVMIR is invalid.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
I have only some minor comments.
@ftynse it may make sense you have a look. The change potentially impacts downstream projects.
mlir/test/Conversion/MemRefToLLVM/convert-dynamic-memref-ops.mlir | ||
---|---|---|
301 | nit: remaining | |
mlir/test/Target/LLVMIR/llvmir-intrinsics.mlir | ||
466 | Should we have an export test with i32 as well? |
nit: remaining