diff mbox series

RISC-V: Add :: for static function calling to avoid confusing

Message ID 20230118031650.70285-1-juzhe.zhong@rivai.ai
State New
Headers show
Series RISC-V: Add :: for static function calling to avoid confusing | expand

Commit Message

juzhe.zhong@rivai.ai Jan. 18, 2023, 3:16 a.m. UTC
From: Ju-Zhe Zhong <juzhe.zhong@rivai.ai>

Since we have same function in class and static function calling get_avl.
Adding :: to avoid codes confusing.

gcc/ChangeLog:

        * config/riscv/riscv-vsetvl.cc (pass_vsetvl::get_backward_fusion_type): Add ::.

---
 gcc/config/riscv/riscv-vsetvl.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jeff Law Jan. 22, 2023, 7:30 p.m. UTC | #1
On 1/17/23 20:16, juzhe.zhong@rivai.ai wrote:
> From: Ju-Zhe Zhong <juzhe.zhong@rivai.ai>
> 
> Since we have same function in class and static function calling get_avl.
> Adding :: to avoid codes confusing.
> 
> gcc/ChangeLog:
> 
>          * config/riscv/riscv-vsetvl.cc (pass_vsetvl::get_backward_fusion_type): Add ::.
Is there any reasonable way to avoid having the same name?  ISTM that's 
the root of the problem and adding "::" is just papering over the real 
issue.

Jeff
diff mbox series

Patch

diff --git a/gcc/config/riscv/riscv-vsetvl.cc b/gcc/config/riscv/riscv-vsetvl.cc
index e11751f00af..b33c198bbd6 100644
--- a/gcc/config/riscv/riscv-vsetvl.cc
+++ b/gcc/config/riscv/riscv-vsetvl.cc
@@ -2073,7 +2073,7 @@  pass_vsetvl::get_backward_fusion_type (const bb_info *bb,
 	    reg = get_vl (insn->rtl ());
 	  else
 	    /* Check AVL operand for vsetvl zero,avl.  */
-	    reg = get_avl (insn->rtl ());
+	    reg = ::get_avl (insn->rtl ());
 	}
     }