Skip to content

Commit dad5db6

Browse files
committed
Corrected slli-srli fusion - the immediate shift ammounts have to be the
same.
1 parent eb79be7 commit dad5db6

File tree

2 files changed

+1
-8
lines changed

2 files changed

+1
-8
lines changed

llvm/lib/Target/RISCV/RISCVFeatures.td

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -997,20 +997,13 @@ def TuneSiFive7 : SubtargetFeature<"sifive7", "RISCVProcFamily", "SiFive7",
997997
[TuneNoDefaultUnroll,
998998
TuneShortForwardBranchOpt]>;
999999

1000-
<<<<<<< HEAD
1001-
def TuneVentanaVeyron : SubtargetFeature<"ventana-veyron", "RISCVProcFamily", "VentanaVeyron",
1002-
"Ventana-Veyron Series processors",
1003-
[TuneLUIADDIFusion]>;
1004-
=======
10051000
def TuneVeyronFusions : SubtargetFeature<"ventana-veyron", "RISCVProcFamily", "VentanaVeyron",
10061001
"Ventana Veyron-Series processors",
10071002
[TuneLUIADDIFusion,
10081003
TuneAUIPCADDIFusion,
10091004
TuneSLLISRLIFusion,
10101005
TuneLDADDFusion]>;
10111006

1012-
>>>>>>> bbf0196a7d42 ([RISCV] Macro-fusion support for veyron-v1 CPU.)
1013-
10141007
// Assume that lock-free native-width atomics are available, even if the target
10151008
// and operating system combination would not usually provide them. The user
10161009
// is responsible for providing any necessary __sync implementations. Code

llvm/lib/Target/RISCV/RISCVMacroFusion.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ static bool isSLLISRLI(const MachineInstr *FirstMI,
8888
return false;
8989

9090
unsigned SLLIImm = FirstMI->getOperand(2).getImm();
91-
if (IsShiftBy48 ? (SLLIImm != 48) : (SLLIImm > 32))
91+
if (IsShiftBy48 ? (SLLIImm != 48) : (SLLIImm != 32))
9292
return false;
9393

9494
return checkRegisters(FirstMI->getOperand(0).getReg(), SecondMI);

0 commit comments

Comments
 (0)