This patch supports to convert the llvm intrinsic to the correspond op. It still leaves some intrinsics to be handle specially.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Thanks!
| mlir/lib/Target/LLVMIR/ConvertFromLLVMIR.cpp | ||
|---|---|---|
| 658 | Please add documentation. | |
| 963 | ||
| mlir/test/Target/LLVMIR/Import/intrinsic.ll | ||
| 12 | Do not pattern-match SSA value names, here and below. We don't actually care about names here so something like %{{.+}} should be sufficient. | |
| 463 | Not: could we have the check line next to the line that is being translated? | |
Thanks for adding this support. I only have some minor comments.
| mlir/include/mlir/Dialect/LLVMIR/LLVMOpBase.td | ||
|---|---|---|
| 281 | I feel like "id" is a little ambiguous, what about "llvmEnumName"? | |
| mlir/lib/Target/LLVMIR/ConvertFromLLVMIR.cpp | ||
| 966 | I think it's better to spell out auto on this line. | |
| 1165 | ditto | |
| mlir/test/Target/LLVMIR/Import/intrinsic.ll | ||
| 3 | Are you intend to add CHECK-s for this function as well? | |
| mlir/lib/Target/LLVMIR/ConvertFromLLVMIR.cpp | ||
|---|---|---|
| 658 | Nit: this returns StringRef, not OperationName. | |
I feel like "id" is a little ambiguous, what about "llvmEnumName"?