about summary refs log tree commit diff
path: root/rt/tst-timer2.c
diff options
context:
space:
mode:
authorMatheus Castanho <msc@linux.ibm.com>2020-03-03 15:20:38 -0300
committerTulio Magno Quites Machado Filho <tuliom@linux.ibm.com>2020-03-06 11:10:12 -0300
commit1c252f0e7e5d78695f19450aa7c470bab445aa8e (patch)
treeff03c7d2393a6eae4cd7029ab44078508e94579e /rt/tst-timer2.c
parentaf09e5e5d9ec3ca20891e61a6922eac984fcbdc4 (diff)
downloadglibc-1c252f0e7e5d78695f19450aa7c470bab445aa8e.tar.gz
glibc-1c252f0e7e5d78695f19450aa7c470bab445aa8e.tar.xz
glibc-1c252f0e7e5d78695f19450aa7c470bab445aa8e.zip
powerpc: Fix feraiseexcept and feclearexcept macros
A recent change to fenvinline.h modified the check if __e is a
a power of 2 inside feraiseexcept and feclearexcept macros.  It
introduced the use of the powerof2 macro but also removed the
if statement checking whether __e != 0 before issuing an mtfsb*
instruction.  This is problematic because powerof2 (0) evaluates
to 1 and without the removed if __e is allowed to be 0 when
__builtin_clz is called.  In that case the value 32 is passed
to __MTFSB*, which is invalid.

This commit uses __builtin_popcount instead of powerof2 to fix this
issue and avoid the extra check for __e != 0.  This was the approach
used by the initial versions of that previous patch.

Reviewed-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
Diffstat (limited to 'rt/tst-timer2.c')
0 files changed, 0 insertions, 0 deletions