This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] powerpc: Fix feraiseexcept and feclearexcept macros



On 04/03/2020 14:47, Florian Weimer wrote:
> * Matheus Castanho:
> 
>>> __builtin_popcount (e) == 1 does not
>>> seem to be unreasonable to express this condition.
>>>
>>
>> Adhemerval and Florian, apart from the discussion to remove or not
>> fenvinline.h altogether, could we push this commit? This header is
>> currently in a broken state and I'd like to fix it for the time being.
> 
> I think the patch is okay, but please also wait for Adhemerval's
> comments.
> 
> I'd also be okay with powerof2 and a zero check.  I agree that we should
> fix this soon.

I am fine with this patch as is, since either solution is fine. I will
probably send a patch to just remove the fenvinline to start the
discussion whether it is worth to keep continuing provide such
optimization.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]