Re: [Mesa-dev] [PATCH] Replace ONE_DIV_LN2 constant with M_LOG2E

2011-05-07 Thread Brian Paul
On Fri, May 6, 2011 at 11:01 AM, Matt Turner wrote: > 1/ln(2) is equivalent to log2(e), so define it as such. > > log2(e) = ln(e)/ln(2) = 1/ln(2) > > Worst of all, the definitions for M_LOG2E and ONE_DIV_LN2 > (right beside each other!) weren't the same. > > Reviewed-by: Alex Deucher > Signed-off

[Mesa-dev] [PATCH] Replace ONE_DIV_LN2 constant with M_LOG2E

2011-05-06 Thread Matt Turner
1/ln(2) is equivalent to log2(e), so define it as such. log2(e) = ln(e)/ln(2) = 1/ln(2) Worst of all, the definitions for M_LOG2E and ONE_DIV_LN2 (right beside each other!) weren't the same. Reviewed-by: Alex Deucher Signed-off-by: Matt Turner --- src/mesa/main/compiler.h |4