This revision starts an Arm-specific ArmSVE dialect discussed in the discourse RFC thread:
https://llvm.discourse.group/t/rfc-vector-dialects-neon-and-sve/2284
Paths
| Differential D92172
[mlir][ArmSVE][RFC] Add an ArmSVE dialect ClosedPublic Authored by jsetoain on Nov 26 2020, 4:13 AM.
Details Summary This revision starts an Arm-specific ArmSVE dialect discussed in the discourse RFC thread: https://llvm.discourse.group/t/rfc-vector-dialects-neon-and-sve/2284
Diff Detail
Event TimelineHerald added subscribers: stephenneuendorffer, nicolasvasilache. · View Herald TranscriptNov 26 2020, 4:13 AM jsetoain retitled this revision from [mlir][SVE] Add an SVE dialect to [mlir][SVE][RFC] Add an SVE dialect.Nov 26 2020, 4:13 AM rriddle added inline comments.
This revision now requires changes to proceed.Nov 26 2020, 4:21 AM Comment Actions
https://llvm.discourse.group/t/rfc-vector-dialects-neon-and-sve/2284 but manually creating http links and synchronizing at sub minute granularity is hard :) @jsetoain could you please update your commit message with this link? Comment Actions
Thanks, I just assumed I missed the thread that discussed this. jsetoain marked an inline comment as done. Comment ActionsAdded missing documentation for SVE intrinsics jsetoain added inline comments.
Comment Actions Fixed issue with doc formating.
jsetoain added inline comments.
jsetoain retitled this revision from [mlir][SVE][RFC] Add an SVE dialect to [mlir][ArmSVE][RFC] Add an SVE dialect.Dec 3 2020, 11:03 AM Comment Actions Once my comments are addressed I am happy to land this.
Comment Actions Fixed lint problems. jsetoain added inline comments.
jsetoain marked 2 inline comments as done. Comment ActionsImprove documentation around signedness of operations. Comment Actions Rebased on top of D92171 jsetoain retitled this revision from [mlir][ArmSVE][RFC] Add an SVE dialect to [mlir][ArmSVE][RFC] Add an ArmSVE dialect.Dec 11 2020, 12:24 PM Comment Actions LGTM, but leaving the approval to River since he added some blocking feedback earlier, so I want to make sure he feels everything has been addressed
rriddle added inline comments.
This revision is now accepted and ready to land.Dec 14 2020, 10:41 AM Comment Actions Reorder operations and initializations alphabetically jsetoain added inline comments.
This revision was landed with ongoing or failed builds.Dec 14 2020, 1:35 PM Closed by commit rGaece4e2793cc: [mlir][ArmSVE][RFC] Add an ArmSVE dialect (authored by jsetoain, committed by mehdi_amini). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 307827 mlir/include/mlir/Conversion/Passes.h
mlir/include/mlir/Conversion/Passes.td
mlir/include/mlir/Conversion/SVEToLLVM/ConvertSVEToLLVM.h
mlir/include/mlir/Dialect/CMakeLists.txt
mlir/include/mlir/Dialect/LLVMIR/CMakeLists.txt
mlir/include/mlir/Dialect/LLVMIR/LLVMSVE.td
mlir/include/mlir/Dialect/LLVMIR/LLVMSVEDialect.h
mlir/include/mlir/Dialect/SVE/SVE.td
mlir/include/mlir/Dialect/SVE/SVEDialect.h
mlir/include/mlir/InitAllDialects.h
mlir/include/mlir/InitAllTranslations.h
mlir/lib/Conversion/CMakeLists.txt
mlir/lib/Conversion/PassDetail.h
mlir/lib/Conversion/SVEToLLVM/CMakeLists.txt
mlir/lib/Conversion/SVEToLLVM/ConvertSVEToLLVM.cpp
mlir/lib/Dialect/CMakeLists.txt
mlir/lib/Dialect/LLVMIR/CMakeLists.txt
mlir/lib/Dialect/LLVMIR/IR/LLVMSVEDialect.cpp
mlir/lib/Dialect/SVE/CMakeLists.txt
mlir/lib/Dialect/SVE/IR/SVEDialect.cpp
mlir/lib/Target/CMakeLists.txt
mlir/lib/Target/LLVMIR/LLVMSVEIntr.cpp
mlir/test/Conversion/SVEToLLVM/convert-to-llvm.mlir
mlir/test/Dialect/SVE/roundtrip.mlir
mlir/test/Target/sve.mlir
|
@aartbik improved the conversion to drop one-off passes.
See the changes to https://reviews.llvm.org/D92171 after I rebased on those.
Basically the ConvertVectorToLLVMPass gains a target-specific bit and conditionally includes the dependent dialects as well as the relevant conversion patterns.