RFR: 8282365: Consolidate and improve division by constant idealizations [v39]
    Quan Anh Mai 
    qamai at openjdk.org
       
    Sun Dec 24 00:12:11 UTC 2023
    
    
  
On Mon, 18 Dec 2023 17:13:48 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
>> Quan Anh Mai has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   missing revert
>
> test/hotspot/gtest/opto/test_constant_division.cpp line 66:
> 
>> 64: 
>> 65: template <class T>
>> 66: void magic_divide_constants(T d, T N_neg, T N_pos, juint min_s, T& c, bool& c_ovf, juint& s);
> 
> I don't see any tests here of magic_divide_constants_round_down.
I took the formula quite literally from the paper so I don't think there is a need for a separate test for those cases. It is also covered in the transformation tests from the Java side.

-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/9947#discussion_r1435743303
    
    
More information about the hotspot-compiler-dev
mailing list