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.