[PATCH] target/hexagon: Remove unused is_finite()

Marco Liebel posted 1 patch 1 week, 4 days ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260126223439.46974-2-marco.liebel@oss.qualcomm.com
Maintainers: Brian Cain <brian.cain@oss.qualcomm.com>
target/hexagon/fma_emu.h | 5 -----
1 file changed, 5 deletions(-)
[PATCH] target/hexagon: Remove unused is_finite()
Posted by Marco Liebel 1 week, 4 days ago
is_finite() is defined but never called anywhere in the codebase.

Signed-off-by: Marco Liebel <marco.liebel@oss.qualcomm.com>
---
 target/hexagon/fma_emu.h | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/target/hexagon/fma_emu.h b/target/hexagon/fma_emu.h
index fed054b609..910530fff8 100644
--- a/target/hexagon/fma_emu.h
+++ b/target/hexagon/fma_emu.h
@@ -18,11 +18,6 @@
 #ifndef HEXAGON_FMA_EMU_H
 #define HEXAGON_FMA_EMU_H
 
-static inline bool is_finite(float64 x)
-{
-    return !float64_is_any_nan(x) && !float64_is_infinity(x);
-}
-
 int32_t float64_getexp(float64 f64);
 static inline uint32_t float32_getexp_raw(float32 f32)
 {
-- 
2.43.0
Re: [PATCH] target/hexagon: Remove unused is_finite()
Posted by Taylor Simpson 1 week, 3 days ago
On Mon, Jan 26, 2026 at 3:35 PM Marco Liebel <marco.liebel@oss.qualcomm.com>
wrote:

> is_finite() is defined but never called anywhere in the codebase.
>
> Signed-off-by: Marco Liebel <marco.liebel@oss.qualcomm.com>
> ---
>  target/hexagon/fma_emu.h | 5 -----
>  1 file changed, 5 deletions(-)
>

Reviewed-by: Taylor Simpson <ltaylorsimpson@gmail.com>
Re: [PATCH] target/hexagon: Remove unused is_finite()
Posted by Brian Cain 1 week, 3 days ago
On Tue, Jan 27, 2026 at 1:09 PM Taylor Simpson <ltaylorsimpson@gmail.com> wrote:
>
>
>
> On Mon, Jan 26, 2026 at 3:35 PM Marco Liebel <marco.liebel@oss.qualcomm.com> wrote:
>>
>> is_finite() is defined but never called anywhere in the codebase.
>>
>> Signed-off-by: Marco Liebel <marco.liebel@oss.qualcomm.com>
>> ---
>>  target/hexagon/fma_emu.h | 5 -----
>>  1 file changed, 5 deletions(-)
>
>
> Reviewed-by: Taylor Simpson <ltaylorsimpson@gmail.com>

Reviewed-by: Brian Cain <brian.cain@oss.qualcomm.com>