target/arm: Fix return values in fp_sysreg_checks()
The fp_sysreg_checks() function is supposed to be returning an FPSysRegCheckResult, which is an enum with three possible values. However, three places in the function "return false" (a hangover from a previous iteration of the design where the function just returned a bool). Make these return FPSysRegCheckFailed instead (for no functional change, since both false and FPSysRegCheckFailed are zero). Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Message-id: 20210520152840.24453-6-peter.maydell@linaro.org
This commit is contained in:
parent
7e435b9ea6
commit
300137965d
|
@ -691,16 +691,16 @@ static FPSysRegCheckResult fp_sysreg_checks(DisasContext *s, int regno)
|
||||||
break;
|
break;
|
||||||
case ARM_VFP_FPSCR_NZCVQC:
|
case ARM_VFP_FPSCR_NZCVQC:
|
||||||
if (!arm_dc_feature(s, ARM_FEATURE_V8_1M)) {
|
if (!arm_dc_feature(s, ARM_FEATURE_V8_1M)) {
|
||||||
return false;
|
return FPSysRegCheckFailed;
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
case ARM_VFP_FPCXT_S:
|
case ARM_VFP_FPCXT_S:
|
||||||
case ARM_VFP_FPCXT_NS:
|
case ARM_VFP_FPCXT_NS:
|
||||||
if (!arm_dc_feature(s, ARM_FEATURE_V8_1M)) {
|
if (!arm_dc_feature(s, ARM_FEATURE_V8_1M)) {
|
||||||
return false;
|
return FPSysRegCheckFailed;
|
||||||
}
|
}
|
||||||
if (!s->v8m_secure) {
|
if (!s->v8m_secure) {
|
||||||
return false;
|
return FPSysRegCheckFailed;
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
default:
|
default:
|
||||||
|
|
Loading…
Reference in a new issue