Commit 1e74ae32 authored by Huacai Chen's avatar Huacai Chen

LoongArch: Cleanup __builtin_constant_p() checking for cpu_has_*

In the current configuration, cpu_has_lsx and cpu_has_lasx cannot be
constants. So cleanup the __builtin_constant_p() checking to reduce the
complexity.
Reviewed-by: default avatarWANG Xuerui <git@xen0n.name>
Signed-off-by: default avatarHuacai Chen <chenhuacai@loongson.cn>
parent 4eece7e6
...@@ -218,14 +218,7 @@ static inline void restore_lsx(struct task_struct *t) ...@@ -218,14 +218,7 @@ static inline void restore_lsx(struct task_struct *t)
static inline void init_lsx_upper(void) static inline void init_lsx_upper(void)
{ {
/* if (cpu_has_lsx)
* Check cpu_has_lsx only if it's a constant. This will allow the
* compiler to optimise out code for CPUs without LSX without adding
* an extra redundant check for CPUs with LSX.
*/
if (__builtin_constant_p(cpu_has_lsx) && !cpu_has_lsx)
return;
_init_lsx_upper(); _init_lsx_upper();
} }
...@@ -294,7 +287,7 @@ static inline void restore_lasx_upper(struct task_struct *t) {} ...@@ -294,7 +287,7 @@ static inline void restore_lasx_upper(struct task_struct *t) {}
static inline int thread_lsx_context_live(void) static inline int thread_lsx_context_live(void)
{ {
if (__builtin_constant_p(cpu_has_lsx) && !cpu_has_lsx) if (!cpu_has_lsx)
return 0; return 0;
return test_thread_flag(TIF_LSX_CTX_LIVE); return test_thread_flag(TIF_LSX_CTX_LIVE);
...@@ -302,7 +295,7 @@ static inline int thread_lsx_context_live(void) ...@@ -302,7 +295,7 @@ static inline int thread_lsx_context_live(void)
static inline int thread_lasx_context_live(void) static inline int thread_lasx_context_live(void)
{ {
if (__builtin_constant_p(cpu_has_lasx) && !cpu_has_lasx) if (!cpu_has_lasx)
return 0; return 0;
return test_thread_flag(TIF_LASX_CTX_LIVE); return test_thread_flag(TIF_LASX_CTX_LIVE);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment