[llvm] [RISCV][GISEL] Do not initialize GlobalISel objects unless needed (PR #98233)

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 9 15:55:34 PDT 2024


================
@@ -1330,7 +1330,7 @@ void RISCVInstructionSelector::emitFence(AtomicOrdering FenceOrdering,
 namespace llvm {
 InstructionSelector *
 createRISCVInstructionSelector(const RISCVTargetMachine &TM,
-                               RISCVSubtarget &Subtarget,
+                               const RISCVSubtarget &Subtarget,
----------------
topperc wrote:

Make RBI const too.

The targets inconsistent about whether these arguments are const reference or not. Maybe we should fix them all?

https://github.com/llvm/llvm-project/pull/98233


More information about the llvm-commits mailing list