RFR: 8327964: Simplify BigInteger.implMultiplyToLen intrinsic [v5]
    Dean Long 
    dlong at openjdk.org
       
    Wed Apr 17 20:08:00 UTC 2024
    
    
  
On Wed, 17 Apr 2024 12:35:54 GMT, Yudi Zheng <yzheng at openjdk.org> wrote:
>> Moving array construction within BigInteger.implMultiplyToLen intrinsic candidate to its caller simplifies the intrinsic implementation in JIT compiler.
>
> Yudi Zheng has updated the pull request incrementally with one additional commit since the last revision:
> 
>   address comment.
src/hotspot/cpu/x86/macroAssembler_x86.cpp line 6662:
> 6660:   push(tmp5);
> 6661: 
> 6662:   push(xlen);
There may be an opportunity here (separate RFE?) to get rid of the save/restore for these.  I don't think it's necessary if this is called as part of a C2 stub.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18226#discussion_r1569452818
    
    
More information about the hotspot-compiler-dev
mailing list