[llvm] [BOLT] Fix warnings (PR #136172)

Kazu Hirata via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 17 10:45:59 PDT 2025


https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/136172

This patch fixes:

  bolt/lib/Target/RISCV/RISCVMCPlusBuilder.cpp:558:23: error:
  '(anonymous namespace)::RISCVMCPlusBuilder::createCmpJE' hides
  overloaded virtual function [-Werror,-Woverloaded-virtual]

  bolt/lib/Target/RISCV/RISCVMCPlusBuilder.cpp:780:8: error:
  '(anonymous
  namespace)::RISCVMCPlusBuilder::convertIndirectCallToLoad' hides
  overloaded virtual function [-Werror,-Woverloaded-virtual]


>From f6a2d7535168f22ed383d366237fc8289b050101 Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Thu, 17 Apr 2025 09:56:42 -0700
Subject: [PATCH] [BOLT] Fix warnings

This patch fixes:

  bolt/lib/Target/RISCV/RISCVMCPlusBuilder.cpp:558:23: error:
  '(anonymous namespace)::RISCVMCPlusBuilder::createCmpJE' hides
  overloaded virtual function [-Werror,-Woverloaded-virtual]

  bolt/lib/Target/RISCV/RISCVMCPlusBuilder.cpp:780:8: error:
  '(anonymous
  namespace)::RISCVMCPlusBuilder::convertIndirectCallToLoad' hides
  overloaded virtual function [-Werror,-Woverloaded-virtual]
---
 bolt/lib/Target/RISCV/RISCVMCPlusBuilder.cpp | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/bolt/lib/Target/RISCV/RISCVMCPlusBuilder.cpp b/bolt/lib/Target/RISCV/RISCVMCPlusBuilder.cpp
index 0e27d29019e95..ec6aa8b3c9415 100644
--- a/bolt/lib/Target/RISCV/RISCVMCPlusBuilder.cpp
+++ b/bolt/lib/Target/RISCV/RISCVMCPlusBuilder.cpp
@@ -555,9 +555,9 @@ class RISCVMCPlusBuilder : public MCPlusBuilder {
                .addReg(RegCnt);
   }
 
-  InstructionListType createCmpJE(MCPhysReg RegNo, MCPhysReg RegTmp,
-                                  const MCSymbol *Target,
-                                  MCContext *Ctx) const {
+  InstructionListType createCmpJERISCV(MCPhysReg RegNo, MCPhysReg RegTmp,
+                                       const MCSymbol *Target,
+                                       MCContext *Ctx) const {
     InstructionListType Insts;
     Insts.emplace_back(
         MCInstBuilder(RISCV::SUB).addReg(RegTmp).addReg(RegNo).addReg(RegNo));
@@ -718,7 +718,7 @@ class RISCVMCPlusBuilder : public MCPlusBuilder {
     Insts.emplace_back();
     loadReg(Insts.back(), RISCV::X10, RISCV::X10, 0);
     InstructionListType cmpJmp =
-        createCmpJE(RISCV::X10, RISCV::X11, IndCallHandler, Ctx);
+        createCmpJERISCV(RISCV::X10, RISCV::X11, IndCallHandler, Ctx);
     Insts.insert(Insts.end(), cmpJmp.begin(), cmpJmp.end());
     Insts.emplace_back();
     createStackPointerIncrement(Insts.back(), 16);
@@ -777,8 +777,8 @@ class RISCVMCPlusBuilder : public MCPlusBuilder {
     return createGetter(Ctx, "__bolt_instr_num_funcs");
   }
 
-  void convertIndirectCallToLoad(MCInst &Inst, MCPhysReg Reg,
-                                 MCPhysReg ZeroReg) const {
+  void convertIndirectCallToLoadRISCV(MCInst &Inst, MCPhysReg Reg,
+                                      MCPhysReg ZeroReg) const {
     bool IsTailCall = isTailCall(Inst);
     if (IsTailCall)
       removeAnnotation(Inst, MCPlus::MCAnnotation::kTailCall);
@@ -845,7 +845,7 @@ class RISCVMCPlusBuilder : public MCPlusBuilder {
     InstructionListType Insts;
     spillRegs(Insts, {RISCV::X10, RISCV::X11});
     Insts.emplace_back(CallInst);
-    convertIndirectCallToLoad(Insts.back(), RISCV::X10, RISCV::X0);
+    convertIndirectCallToLoadRISCV(Insts.back(), RISCV::X10, RISCV::X0);
     InstructionListType LoadImm = createLoadImmediate(RISCV::X11, CallSiteID);
     Insts.insert(Insts.end(), LoadImm.begin(), LoadImm.end());
     spillRegs(Insts, {RISCV::X10, RISCV::X11});



More information about the llvm-commits mailing list