Re: RFR: 8355300: Add final to BitSieve [v2]

2025-04-22 Thread Per Minborg
On Tue, 22 Apr 2025 15:07:29 GMT, Shaojin Wen wrote: >> As the title says, adding final to fields that do not change and BitSieve >> that will not be inherited will make C2 optimization more friendly. > > Shaojin Wen has updated the pull request incrementally with one additional > commit since

Re: RFR: 8355300: Add final to BitSieve [v2]

2025-04-22 Thread Chen Liang
On Tue, 22 Apr 2025 15:04:07 GMT, Shaojin Wen wrote: >> As the title says, adding final to fields that do not change and BitSieve >> that will not be inherited will make C2 optimization more friendly. > > Shaojin Wen has updated the pull request incrementally with one additional > commit since

Re: RFR: 8355300: Add final to BitSieve [v2]

2025-04-22 Thread Shaojin Wen
> As the title says, adding final to fields that do not change and BitSieve > that will not be inherited will make C2 optimization more friendly. Shaojin Wen has updated the pull request incrementally with one additional commit since the last revision: Update src/java.base/share/classes/java/

Re: RFR: 8355300: Add final to BitSieve

2025-04-22 Thread Chen Liang
On Tue, 22 Apr 2025 11:10:46 GMT, Shaojin Wen wrote: > As the title says, adding final to fields that do not change and BitSieve > that will not be inherited will make C2 optimization more friendly. src/java.base/share/classes/java/math/BitSieve.java line 51: > 49: * Stores the bits in th