Add intrinsics for the following operations:
- eor3, bcax
- bsl, bsl1n, bsl2n, nbsl
Paths
| Differential D74785
[AArch64][SVE] Add intrinsics for SVE2 bitwise ternary operations ClosedPublic Authored by dancgr on Feb 18 2020, 12:03 PM.
Details Summary Add intrinsics for the following operations:
Diff Detail
Event Timeline
This revision is now accepted and ready to land.Feb 19 2020, 2:31 PM Closed by commit rGce70e2899879: [AArch64][SVE] Add intrinsics for SVE2 bitwise ternary operations (authored by dancgr). · Explain WhyFeb 20 2020, 8:30 AM This revision was automatically updated to reflect the committed changes. Comment Actions Hi, I think this might've caused some test failures on our bots and others on buildbot: FAIL: LLVM :: MC/AArch64/SVE2/bsl-diagnostics.s (23685 of 36141) ******************** TEST 'LLVM :: MC/AArch64/SVE2/bsl-diagnostics.s' FAILED ******************** Script: -- : 'RUN: at line 1'; not /b/s/w/ir/k/recipe_cleanup/clangOZ9WCk/llvm_build_dir/bin/llvm-mc -triple=aarch64 -show-encoding -mattr=+sve2 2>&1 < /b/s/w/ir/k/llvm-project/llvm/test/MC/AArch64/SVE2/bsl-diagnostics.s| /b/s/w/ir/k/recipe_cleanup/clangOZ9WCk/llvm_build_dir/bin/FileCheck /b/s/w/ir/k/llvm-project/llvm/test/MC/AArch64/SVE2/bsl-diagnostics.s -- Exit Code: 1 Command Output (stderr): -- /b/s/w/ir/k/llvm-project/llvm/test/MC/AArch64/SVE2/bsl-diagnostics.s:8:11: error: CHECK: expected string not found in input // CHECK: [[@LINE-1]]:{{[0-9]+}}: error: invalid element width ^ <stdin>:1:1: note: scanning from here <stdin>:26:11: error: operand must match destination register ^ <stdin>:1:1: note: with "@LINE-1" equal to "7" <stdin>:26:11: error: operand must match destination register ^ <stdin>:1:2: note: possible intended match here <stdin>:26:11: error: operand must match destination register ^ ... Testing Time: 135.11s ******************** Failing Tests (4): LLVM :: MC/AArch64/SVE2/bsl-diagnostics.s LLVM :: MC/AArch64/SVE2/bsl1n-diagnostics.s LLVM :: MC/AArch64/SVE2/bsl2n-diagnostics.s LLVM :: MC/AArch64/SVE2/nbsl-diagnostics.s Could you send out a fix or revert? Thanks. Builder: https://ci.chromium.org/p/fuchsia/builders/ci/clang-linux-x64/b8887916992587484480 Comment Actions I reverted this in 6f4d9d10293d8e93d7cf3397e3a82ac19c8b629f due to breaking tests everywhere. Comment Actions Will look into that and update this review with the necessary changes. Thanks @thakis for reverting the commit. This revision is now accepted and ready to land.Feb 21 2020, 8:33 AM Closed by commit rGdb9c40f5624e: [AArch64][SVE] Add intrinsics for SVE2 bitwise ternary operations (authored by dancgr). · Explain WhyFeb 21 2020, 9:19 AM This revision was automatically updated to reflect the committed changes. Comment Actions Latest commit adds a change to the failing MC tests to be exactly the same as the MC tests for eor3 and bcax.
Revision Contents
Diff 245892 llvm/include/llvm/IR/IntrinsicsAArch64.td
llvm/lib/Target/AArch64/AArch64SVEInstrInfo.td
llvm/lib/Target/AArch64/SVEInstrFormats.td
llvm/test/CodeGen/AArch64/sve2-bitwise-ternary.ll
llvm/test/MC/AArch64/SVE2/bsl-diagnostics.s
llvm/test/MC/AArch64/SVE2/bsl1n-diagnostics.s
llvm/test/MC/AArch64/SVE2/bsl2n-diagnostics.s
llvm/test/MC/AArch64/SVE2/nbsl-diagnostics.s
|
Missing newline
Any particular reason you're only testing bsl with nxv2i64 operands?