diff options
author | Bibo Mao <maobibo@loongson.cn> | 2025-03-20 11:21:57 +0800 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2025-03-20 07:58:19 +0100 |
commit | daf78a9d51402e8f70d89f86a8c5f12d02aee667 (patch) | |
tree | a2e79b7bbc85052a4107c1d5e3cf91cec05719ed | |
parent | 0973b505fadedd208e4c3fa1698ffa0b2293006c (diff) | |
download | qemu-daf78a9d51402e8f70d89f86a8c5f12d02aee667.zip qemu-daf78a9d51402e8f70d89f86a8c5f12d02aee667.tar.gz qemu-daf78a9d51402e8f70d89f86a8c5f12d02aee667.tar.bz2 |
target/loongarch: Remove unnecessary temporary variable assignment
Temporary variable ret is assigned at last line and return, it can
be removed and return directly.
Signed-off-by: Bibo Mao <maobibo@loongson.cn>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-ID: <20250320032158.1762751-6-maobibo@loongson.cn>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
-rw-r--r-- | target/loongarch/tcg/tlb_helper.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/target/loongarch/tcg/tlb_helper.c b/target/loongarch/tcg/tlb_helper.c index 646dbf5..182881a 100644 --- a/target/loongarch/tcg/tlb_helper.c +++ b/target/loongarch/tcg/tlb_helper.c @@ -543,7 +543,7 @@ target_ulong helper_lddir(CPULoongArchState *env, target_ulong base, target_ulong level, uint32_t mem_idx) { CPUState *cs = env_cpu(env); - target_ulong badvaddr, index, phys, ret; + target_ulong badvaddr, index, phys; uint64_t dir_base, dir_width; if (unlikely((level == 0) || (level > 4))) { @@ -571,8 +571,7 @@ target_ulong helper_lddir(CPULoongArchState *env, target_ulong base, get_dir_base_width(env, &dir_base, &dir_width, level); index = (badvaddr >> dir_base) & ((1 << dir_width) - 1); phys = base | index << 3; - ret = ldq_phys(cs->as, phys) & TARGET_PHYS_MASK; - return ret; + return ldq_phys(cs->as, phys) & TARGET_PHYS_MASK; } void helper_ldpte(CPULoongArchState *env, target_ulong base, target_ulong odd, |