Re: RFR: 8238669: Long.divideUnsigned is extremely slow for certain values (Needs to be Intrinsic) [v2]

2020-09-09 Thread rgiulietti
On Wed, 9 Sep 2020 06:25:40 GMT, Aleksey Shipilev wrote: >> rgiulietti has updated the pull request incrementally with one additional >> commit since the last revision: >> >> 8238669: Long.divideUnsigned is extremely slow for certain values (Needs >> to

Re: RFR: 8238669: Long.divideUnsigned is extremely slow for certain values (Needs to be Intrinsic) [v2]

2020-09-08 Thread rgiulietti
On Tue, 8 Sep 2020 20:42:41 GMT, rgiulietti wrote: >> src/java.base/share/classes/java/lang/Long.java line 1697: >> >>> 1695: final long q = (dividend >>> 1) / divisor << 1; >>> 1696: final long r = dividend - q * divisor; &g

Re: RFR: 8238669: Long.divideUnsigned is extremely slow for certain values (Needs to be Intrinsic) [v2]

2020-09-08 Thread rgiulietti
net/pipermail/core-libs-dev/2020-September/068474.html rgiulietti has updated the pull request incrementally with one additional commit since the last revision: 8238669: Long.divideUnsigned is extremely slow for certain values (Needs to be Intrinsic) Added helpful parentheses and c

Re: RFR: 8238669: Long.divideUnsigned is extremely slow for certain values (Needs to be Intrinsic)

2020-09-08 Thread rgiulietti
On Tue, 8 Sep 2020 20:30:36 GMT, Éamonn McManus wrote: >> This is a follow-up of the Mercurial-based workflow initiated on the >> core-lib-devs mailing list [0]. Not sure if this >> one is strictly necessary or if the patches sent on the list are sufficient. >> Anyway, I exploit this PR as a

RFR: 8238669: Long.divideUnsigned is extremely slow for certain values (Needs to be Intrinsic)

2020-09-07 Thread rgiulietti
This is a follow-up of the Mercurial-based workflow initiated on the core-lib-devs mailing list [0]. Not sure if this one is strictly necessary or if the patches sent on the list are sufficient. Anyway, I exploit this PR as a test ;-) [0]