]> git.ipfire.org Git - thirdparty/gcc.git/commit
Fold ctz(-x) and ctz(abs(x)) as ctz(x) in match.pd.
authorRoger Sayle <roger@nextmovesoftware.com>
Sat, 27 Jul 2024 14:16:19 +0000 (15:16 +0100)
committerThomas Koenig <tkoenig@gcc.gnu.org>
Sun, 28 Jul 2024 17:06:02 +0000 (19:06 +0200)
commit262163238006aa39eff5ebd8283ec7254161269a
tree0d0e29afb765a51703d9922d8343d53f0fe62349
parentf3ed8cd8f72240a31d6591f0156ea7d3258b1736
Fold ctz(-x) and ctz(abs(x)) as ctz(x) in match.pd.

The subject line pretty much says it all; the count-trailing-zeros function
of -X and abs(X) produce the same result as count-trailing-zeros of X.
This transformation eliminates a negation which may potentially overflow
with an equivalent expression that doesn't [much like the analogous
abs(-X) simplification in match.pd].

I'd noticed this -X equivalence, which isn't mentioned in Hacker's Delight,
investigating whether ranger's non_zero_bits can help determine whether
an integer variable may be converted to a floating point type exactly
(without raising FE_INEXACT), but it turns out this observation isn't
novel, as (disappointingly) LLVM already performs this same folding.

2024-07-27  Roger Sayle  <roger@nextmovesoftware.com>
    Andrew Pinski  <quic_apinski@quicinc.com>

gcc/ChangeLog
* match.pd (ctz (-X) => ctz (X)): New simplification.
(ctz (abs (X)) => ctz (X)): Likewise.

gcc/testsuite/ChangeLog
* gcc.dg/fold-ctz-1.c: New test case.
* gcc.dg/fold-ctz-2.c: Likewise.
gcc/match.pd
gcc/testsuite/gcc.dg/fold-ctz-1.c [new file with mode: 0644]
gcc/testsuite/gcc.dg/fold-ctz-2.c [new file with mode: 0644]