This change renames the intrinsics to not have "experimental" in the name.
The autoupgrader will handle legacy intrinsics.
Relevant ML thread: http://lists.llvm.org/pipermail/llvm-dev/2020-April/140729.html
Paths
| Differential D88787
[llvm][mlir] Promote the experimental reduction intrinsics to be first class intrinsics. ClosedPublic Authored by aemerson on Oct 3 2020, 12:27 PM.
Details Summary This change renames the intrinsics to not have "experimental" in the name. The autoupgrader will handle legacy intrinsics. Relevant ML thread: http://lists.llvm.org/pipermail/llvm-dev/2020-April/140729.html
Diff Detail
Event TimelineComment Actions @aartbik @rriddle @nicolasvasilache MLIR also needed updating to use these new intrinsics, please take a look.
Comment Actions
MLIR part LGTM
Comment Actions Remove the scalar input overload for fadd/fmul. Now the signature is simple llvm.reduce.fadd.v4f32 rather than llvm.reduce.fadd.f32.v4f32 Also fix some langref formatting, and fold in the change to use "sequential" instead of "ordered" while I'm there. Comment Actions One last minor comment - anyone else?
Comment Actions LGTM
This revision is now accepted and ready to land.Oct 7 2020, 10:00 AM Comment Actions I'll make those last changes in the commit. Thanks to everyone who's pushed this forward over the last few years with fixes and reviews! This revision was landed with ongoing or failed builds.Oct 7 2020, 10:37 AM Closed by commit rG322d0afd875d: [llvm][mlir] Promote the experimental reduction intrinsics to be first class… (authored by aemerson). · Explain Why This revision was automatically updated to reflect the committed changes. Comment Actions I think there may be a small remaining issue on the llvmir-intrinsics.mlir test on Windows, as seen here: http://lab.llvm.org:8011/#/builders/82/builds/40/steps/7/logs/stdio $ ":" "RUN: at line 1" $ "e:\build_slave\mlir-x64-windows-ninja\build\bin\mlir-translate.exe" "-mlir-to-llvmir" "E:\build_slave\mlir-x64-windows-ninja\llvm-project\mlir\test\Target\llvmir-intrinsics.mlir" # command stderr: Expected an argument of Vector Type UNREACHABLE executed at E:\build_slave\mlir-x64-windows-ninja\llvm-project\llvm\lib\IR\Function.cpp:1167! Comment Actions
Yep, I've just pushed c1247f0e74bff00ab9a896a8132318916f3e84a7 to hopefully fix that. Large DiffThis large diff affects 182 files. Files without inline comments have been collapsed. Expand All Files
Revision Contents
Diff 296236 llvm/docs/LangRef.rst
llvm/lib/IR/AutoUpgrade.cpp
llvm/test/Bitcode/upgrade-vecreduce-intrinsics.ll
llvm/test/Instrumentation/MemorySanitizer/experimental-reduce.ll
|
Underlines for titles need adjustment.