[RISCV] Remove skipFunction from RISCVVMV0Elimination. (#136080)

topperc · web-flow · commit 8d4b19e3d4a8 · 2025-04-17T07:34:38.000-07:00
skipFunction disables the pass for functions with optnone attribute.
This pass is important to correct register allocation so I don't think
it should be disabled.
diff --git a/llvm/lib/Target/RISCV/RISCVVMV0Elimination.cpp b/llvm/lib/Target/RISCV/RISCVVMV0Elimination.cpp
@@ -79,9 +79,6 @@ static bool isVMV0(const MCOperandInfo &MCOI) {
 }
 
 bool RISCVVMV0Elimination::runOnMachineFunction(MachineFunction &MF) {
-  if (skipFunction(MF.getFunction()))
-    return false;
-
   // Skip if the vector extension is not enabled.
   const RISCVSubtarget *ST = &MF.getSubtarget<RISCVSubtarget>();
   if (!ST->hasVInstructions())

Original file line number	Diff line number	Diff line change
`@@ -79,9 +79,6 @@ static bool isVMV0(const MCOperandInfo &MCOI) {`
`79`	`79`	`}`
`80`	`80`
`81`	`81`	`bool RISCVVMV0Elimination::runOnMachineFunction(MachineFunction &MF) {`
`82`		`- if (skipFunction(MF.getFunction()))`
`83`		`- return false;`
`84`		`-`
`85`	`82`	`// Skip if the vector extension is not enabled.`
`86`	`83`	`const RISCVSubtarget *ST = &MF.getSubtarget<RISCVSubtarget>();`
`87`	`84`	`if (!ST->hasVInstructions())`