[PATCH v3 49/51] tcg/optimize: Re-enable sign-mask optimizations

Richard Henderson posted 51 patches 1 year, 1 month ago
[PATCH v3 49/51] tcg/optimize: Re-enable sign-mask optimizations
Posted by Richard Henderson 1 year, 1 month ago
All instances of s_mask have been converted to the new
representation.  We can now re-enable usage.

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
 tcg/optimize.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tcg/optimize.c b/tcg/optimize.c
index 902c219032..607697e19c 100644
--- a/tcg/optimize.c
+++ b/tcg/optimize.c
@@ -1780,7 +1780,7 @@ static bool fold_exts(OptContext *ctx, TCGOp *op)
         g_assert_not_reached();
     }
 
-    if (0 && !type_change && fold_affected_mask(ctx, op, s_mask & ~s_mask_old)) {
+    if (!type_change && fold_affected_mask(ctx, op, s_mask & ~s_mask_old)) {
         return true;
     }
 
@@ -2489,7 +2489,7 @@ static bool fold_sextract(OptContext *ctx, TCGOp *op)
     s_mask = s_mask_old >> pos;
     s_mask |= -1ull << (len - 1);
 
-    if (0 && pos == 0 && fold_affected_mask(ctx, op, s_mask & ~s_mask_old)) {
+    if (pos == 0 && fold_affected_mask(ctx, op, s_mask & ~s_mask_old)) {
         return true;
     }
 
-- 
2.43.0
Re: [PATCH v3 49/51] tcg/optimize: Re-enable sign-mask optimizations
Posted by Pierrick Bouvier 1 year, 1 month ago
On 12/22/24 08:24, Richard Henderson wrote:
> All instances of s_mask have been converted to the new
> representation.  We can now re-enable usage.
> 
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
>   tcg/optimize.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tcg/optimize.c b/tcg/optimize.c
> index 902c219032..607697e19c 100644
> --- a/tcg/optimize.c
> +++ b/tcg/optimize.c
> @@ -1780,7 +1780,7 @@ static bool fold_exts(OptContext *ctx, TCGOp *op)
>           g_assert_not_reached();
>       }
>   
> -    if (0 && !type_change && fold_affected_mask(ctx, op, s_mask & ~s_mask_old)) {
> +    if (!type_change && fold_affected_mask(ctx, op, s_mask & ~s_mask_old)) {
>           return true;
>       }
>   
> @@ -2489,7 +2489,7 @@ static bool fold_sextract(OptContext *ctx, TCGOp *op)
>       s_mask = s_mask_old >> pos;
>       s_mask |= -1ull << (len - 1);
>   
> -    if (0 && pos == 0 && fold_affected_mask(ctx, op, s_mask & ~s_mask_old)) {
> +    if (pos == 0 && fold_affected_mask(ctx, op, s_mask & ~s_mask_old)) {
>           return true;
>       }
>   

Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>