[PATCH] D88545: [ARM] Removed hasSideEffects from signed/unsigned saturates
Meera Nakrani via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 30 02:13:59 PDT 2020
MeeraN created this revision.
MeeraN added reviewers: dmgreen, SjoerdMeijer, samparker.
Herald added subscribers: danielkiss, gbedwell, hiraditya, kristof.beyls.
Herald added a reviewer: andreadb.
Herald added a project: LLVM.
MeeraN requested review of this revision.
Removed hasSideEffects from SSAT and USAT so that they are no longer
marked as unpredictable
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D88545
Files:
llvm/lib/Target/ARM/ARMInstrThumb2.td
llvm/test/tools/llvm-mca/ARM/m4-int.s
Index: llvm/test/tools/llvm-mca/ARM/m4-int.s
===================================================================
--- llvm/test/tools/llvm-mca/ARM/m4-int.s
+++ llvm/test/tools/llvm-mca/ARM/m4-int.s
@@ -746,9 +746,9 @@
# CHECK-NEXT: 1 1 1.00 smulwt r0, r1, r2
# CHECK-NEXT: 1 2 1.00 smusd r0, r1, r2
# CHECK-NEXT: 1 2 1.00 smusdx r0, r1, r2
-# CHECK-NEXT: 1 1 1.00 U ssat r0, #1, r2
-# CHECK-NEXT: 1 1 1.00 U ssat r0, #1, r2, lsl #1
-# CHECK-NEXT: 1 1 1.00 U ssat16 r0, #1, r1
+# CHECK-NEXT: 1 1 1.00 ssat r0, #1, r2
+# CHECK-NEXT: 1 1 1.00 ssat r0, #1, r2, lsl #1
+# CHECK-NEXT: 1 1 1.00 ssat16 r0, #1, r1
# CHECK-NEXT: 1 1 1.00 * * U ssax r0, r1, r2
# CHECK-NEXT: 1 1 1.00 * * U ssbb
# CHECK-NEXT: 1 1 1.00 * * U ssub16 r0, r1, r2
@@ -858,9 +858,9 @@
# CHECK-NEXT: 1 1 1.00 uqsub8 r0, r1, r2
# CHECK-NEXT: 1 1 1.00 usad8 r0, r1, r2
# CHECK-NEXT: 1 1 1.00 usada8 r0, r1, r2, r3
-# CHECK-NEXT: 1 1 1.00 U usat r0, #1, r1
-# CHECK-NEXT: 1 1 1.00 U usat r0, #1, r1, lsl #1
-# CHECK-NEXT: 1 1 1.00 U usat16 r0, #1, r1
+# CHECK-NEXT: 1 1 1.00 usat r0, #1, r1
+# CHECK-NEXT: 1 1 1.00 usat r0, #1, r1, lsl #1
+# CHECK-NEXT: 1 1 1.00 usat16 r0, #1, r1
# CHECK-NEXT: 1 1 1.00 * * U usax r0, r1, r2
# CHECK-NEXT: 1 1 1.00 * * U usub16 r0, r1, r2
# CHECK-NEXT: 1 1 1.00 * * U usub8 r0, r1, r2
Index: llvm/lib/Target/ARM/ARMInstrThumb2.td
===================================================================
--- llvm/lib/Target/ARM/ARMInstrThumb2.td
+++ llvm/lib/Target/ARM/ARMInstrThumb2.td
@@ -2575,7 +2575,6 @@
Requires<[IsThumb2, HasDSP]>;
// Signed/Unsigned saturate.
-let hasSideEffects = 1 in
class T2SatI<dag iops, string opc, string asm>
: T2I<(outs rGPR:$Rd), iops, NoItinerary, opc, asm, []> {
bits<4> Rd;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88545.295208.patch
Type: text/x-patch
Size: 2511 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200930/be319afd/attachment.bin>
More information about the llvm-commits
mailing list