[llvm] [AVR] Remove extra ROL / ROR operands (PR #117510)
Sergei Barannikov via llvm-commits
llvm-commits at lists.llvm.org
Sun Nov 24 15:55:49 PST 2024
https://github.com/s-barannikov created https://github.com/llvm/llvm-project/pull/117510
The nodes have one input, shift amount of 1 is implied.
>From ff6a9211d97c476881f81b8a9a823bc446bf50b9 Mon Sep 17 00:00:00 2001
From: Sergei Barannikov <barannikov88 at gmail.com>
Date: Mon, 25 Nov 2024 02:47:23 +0300
Subject: [PATCH] [AVR] Remove extra ROL / ROR operands
The nodes have one input, shift amount of 1 is implied.
---
llvm/lib/Target/AVR/AVRISelLowering.cpp | 12 ++++--------
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/llvm/lib/Target/AVR/AVRISelLowering.cpp b/llvm/lib/Target/AVR/AVRISelLowering.cpp
index 1737ec7b671992..07c79f6f227b02 100644
--- a/llvm/lib/Target/AVR/AVRISelLowering.cpp
+++ b/llvm/lib/Target/AVR/AVRISelLowering.cpp
@@ -412,24 +412,20 @@ SDValue AVRTargetLowering::LowerShifts(SDValue Op, SelectionDAG &DAG) const {
} else if (Op.getOpcode() == ISD::ROTL && ShiftAmount == 3) {
// Optimize left rotation 3 bits to swap then right rotation 1 bit.
Victim = DAG.getNode(AVRISD::SWAP, dl, VT, Victim);
- Victim =
- DAG.getNode(AVRISD::ROR, dl, VT, Victim, DAG.getConstant(1, dl, VT));
+ Victim = DAG.getNode(AVRISD::ROR, dl, VT, Victim);
ShiftAmount = 0;
} else if (Op.getOpcode() == ISD::ROTR && ShiftAmount == 3) {
// Optimize right rotation 3 bits to swap then left rotation 1 bit.
Victim = DAG.getNode(AVRISD::SWAP, dl, VT, Victim);
- Victim =
- DAG.getNode(AVRISD::ROL, dl, VT, Victim, DAG.getConstant(1, dl, VT));
+ Victim = DAG.getNode(AVRISD::ROL, dl, VT, Victim);
ShiftAmount = 0;
} else if (Op.getOpcode() == ISD::ROTL && ShiftAmount == 7) {
// Optimize left rotation 7 bits to right rotation 1 bit.
- Victim =
- DAG.getNode(AVRISD::ROR, dl, VT, Victim, DAG.getConstant(1, dl, VT));
+ Victim = DAG.getNode(AVRISD::ROR, dl, VT, Victim);
ShiftAmount = 0;
} else if (Op.getOpcode() == ISD::ROTR && ShiftAmount == 7) {
// Optimize right rotation 7 bits to left rotation 1 bit.
- Victim =
- DAG.getNode(AVRISD::ROL, dl, VT, Victim, DAG.getConstant(1, dl, VT));
+ Victim = DAG.getNode(AVRISD::ROL, dl, VT, Victim);
ShiftAmount = 0;
} else if ((Op.getOpcode() == ISD::ROTR || Op.getOpcode() == ISD::ROTL) &&
ShiftAmount >= 4) {
More information about the llvm-commits
mailing list