diff options
author | Gabriel F. T. Gomes <gftg@linux.vnet.ibm.com> | 2016-01-22 18:05:05 -0200 |
---|---|---|
committer | Tulio Magno Quites Machado Filho <tuliom@linux.vnet.ibm.com> | 2016-03-08 15:30:28 -0300 |
commit | 183a34dc4a07cad723cd78e494d16a5ef0b6a63e (patch) | |
tree | ddbc585df379835c6347b72cf6c3ebf9d781fb16 /ChangeLog | |
parent | 911569d02dec023d949d96aa7b0e828c91c06f55 (diff) | |
download | glibc-183a34dc4a07cad723cd78e494d16a5ef0b6a63e.tar glibc-183a34dc4a07cad723cd78e494d16a5ef0b6a63e.tar.gz glibc-183a34dc4a07cad723cd78e494d16a5ef0b6a63e.tar.bz2 glibc-183a34dc4a07cad723cd78e494d16a5ef0b6a63e.zip |
powerpc: Remove uses of operand modifier (%s) in inline asm
The operand modifier %s on powerpc is an undocumented internal implementation
detail of GCC. Besides that, the GCC community wants to remove it. This patch
rewrites the expressions that use this modifier with logically equivalent
expressions that don't require it.
Explanation for the substitution:
The %s modifier takes an immediate operand and prints 32 less such immediate.
Thus, in the previous code, the expression resulted in:
32 - __builtin_ffs(e)
where e was guaranteed to have exactly a single bit set, by the following
expressions:
(e & (e-1) == 0) : e has at most one bit set.
(e != 0) : e is not zero, thus it has at least one bit set.
Since we guarantee that there is exactly only one bit set, the following
statement is true:
32 - __builtin_ffs(e) == __builtin_clz(e)
Thus, we can replace __builtin_ffs with __builtin_clz and remove the %s operand
modifier.
Diffstat (limited to 'ChangeLog')
-rw-r--r-- | ChangeLog | 6 |
1 files changed, 6 insertions, 0 deletions
@@ -1,3 +1,9 @@ +2016-03-08 Gabriel F. T. Gomes <gftg@linux.vnet.ibm.com> + + * sysdeps/powerpc/bits/fenvinline.h (feraiseexcept): Remove use of %s + operand modifier. + (feclearexcept): Likewise. + 2016-03-08 Carlos Eduardo Seo <cseo@linux.vnet.ibm.com> * sysdeps/powerpc/dl-procinfo.c (_dl_powerpc_cap_flags): Updated |