[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 1/2] target/mips: Remove identical if/else branches
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v2 1/2] target/mips: Remove identical if/else branches |
Date: |
Tue, 30 Jun 2020 19:22:55 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0 |
On 6/30/20 6:46 PM, Aleksandar Markovic wrote:
> Remove the segment:
>
> if (other_tc == other->current_tc) {
> tccause = other->CP0_Cause;
> } else {
> tccause = other->CP0_Cause;
> }
>
> Original contributor can't remember what was his intention.
>
> Bug: 1885718
The format documented in the wiki is:
Fixes: 5a25ce9487 ("mips: Hook in more reg accesses via mttr/mftr")
Buglink: https://bugs.launchpad.net/qemu/+bug/1885718
See:
https://wiki.qemu.org/Contribute/SubmitAPatch#Write_a_meaningful_commit_message
> Signed-off-by: Aleksandar Markovic <aleksandar.qemu.devel@gmail.com>
> ---
> target/mips/cp0_helper.c | 9 +--------
> 1 file changed, 1 insertion(+), 8 deletions(-)
>
> diff --git a/target/mips/cp0_helper.c b/target/mips/cp0_helper.c
> index bbf12e4a97..de64add038 100644
> --- a/target/mips/cp0_helper.c
> +++ b/target/mips/cp0_helper.c
> @@ -375,16 +375,9 @@ target_ulong helper_mftc0_entryhi(CPUMIPSState *env)
> target_ulong helper_mftc0_cause(CPUMIPSState *env)
> {
> int other_tc = env->CP0_VPEControl & (0xff << CP0VPECo_TargTC);
> - int32_t tccause;
> CPUMIPSState *other = mips_cpu_map_tc(env, &other_tc);
>
> - if (other_tc == other->current_tc) {
> - tccause = other->CP0_Cause;
> - } else {
> - tccause = other->CP0_Cause;
> - }
> -
> - return tccause;
> + return other->CP0_Cause;
> }
>
> target_ulong helper_mftc0_status(CPUMIPSState *env)
>