On Tue, 3 Sep 2024 22:55:18 GMT, Joe Darcy <da...@openjdk.org> wrote:

>> Srinivas Vamsi Parasa has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Add stub initialization and extra tanh tests
>
> test/jdk/java/lang/Math/HyperbolicTests.java line 984:
> 
>> 982:         double b1 = 0.02;
>> 983:         double b2 = 5.1;
>> 984:         double b3 = 55 * Math.log(2)/2; // ~19.062
> 
> Probably better to use StrictMath.log here or, better use, precompute the 
> value as a constant and document its conceptual origin.

Please see the updated code which uses the precomputed value of `b3`.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/20657#discussion_r1746031432

Reply via email to