arch/x86: remove the redundant check

Taemin Ha · avagin · commit 9e05b658763f · 2023-10-08T08:47:13.000-07:00
The is_native field is a boolean. Therefore, else if() should can be
changed to a simple else{}.

Signed-off-by: Taemin Ha &lt;taeminha@cs.utexas.edu&gt;
Signed-off-by: Andrei Vagin &lt;avagin@gmail.com&gt;
diff --git a/criu/arch/x86/sigframe.c b/criu/arch/x86/sigframe.c
@@ -23,7 +23,7 @@ int sigreturn_prep_fpu_frame(struct rt_sigframe *sigframe, struct rt_sigframe *r
 		}
 
 		sigframe->native.uc.uc_mcontext.fpstate = (uint64_t)addr;
-	} else if (!sigframe->is_native) {
+	} else {
 		unsigned long addr = (unsigned long)(void *)&fpu_state->fpu_state_ia32.xsave;
 		sigframe->compat.uc.uc_mcontext.fpstate = (uint32_t)(unsigned long)(void *)&fpu_state->fpu_state_ia32;
 		if ((addr % 64ul)) {

Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,7 @@ int sigreturn_prep_fpu_frame(struct rt_sigframe sigframe, struct rt_sigframe r`
`23`	`23`	`}`
`24`	`24`
`25`	`25`	`sigframe->native.uc.uc_mcontext.fpstate = (uint64_t)addr;`
`26`		`- } else if (!sigframe->is_native) {`
	`26`	`+ } else {`
`27`	`27`	`unsigned long addr = (unsigned long)(void *)&fpu_state->fpu_state_ia32.xsave;`
`28`	`28`	`sigframe->compat.uc.uc_mcontext.fpstate = (uint32_t)(unsigned long)(void *)&fpu_state->fpu_state_ia32;`
`29`	`29`	`if ((addr % 64ul)) {`