On 17/6/26 16:51, Richard Henderson wrote:
On 6/16/26 12:17, Philippe Mathieu-Daudé wrote:case DIAG308_STORE: case DIAG308_PV_STORE: if (diag308_parm_check(env, r1, addr, ra, true)) { - return; + break; } if (subcode == DIAG308_PV_STORE) { iplb = s390_ipl_get_iplb_pv();Why change these return to break, rather than just return false? There aren't *that* many places where reset_requested are set.
I thought the diff would result smaller, but not really ;) - 4 files changed, 25 insertions(+), 18 deletions(-) + 4 files changed, 22 insertions(+), 23 deletions(-)
