On Wed, 18 May 2022 14:59:49 GMT, Quan Anh Mai wrote:
>> Hi,
>>
>> This patch optimises the matching rules for floating-point comparison with
>> respects to eq/ne on x86-64
>>
>> 1, When the inputs of a comparison is the same (i.e `isNaN` patterns), `ZF`
>> is always set, so we don't need
On Mon, 11 Apr 2022 02:01:17 GMT, David Holmes wrote:
> This change appears to be causing crashes in tier4 - possibly Xcomp related:
>
> # assert(ctrl == kit.control()) failed: Control flow was added although the
> intrinsic bailed out
> I will file a new bug.
Thank you for informing this
On Wed, 6 Apr 2022 06:23:47 GMT, Jatin Bhateja wrote:
>>> Also need a jtreg test for this.
>>
>> Thanks Sandhya for the review. Made the suggested changes and added jtreg
>> tests as well.
>
> Hi @vamsi-parasa , thanks for addressing my comments, looks good to me
> otherwise apart from the
On Tue, 22 Feb 2022 09:24:47 GMT, Srinivas Vamsi Parasa
wrote:
> Optimizes the divideUnsigned() and remainderUnsigned() methods in
> java.lang.Integer and java.lang.Long classes using x86 intrinsics. This
> change shows 3x improvement for Integer methods and upto 25% improvement fo
On Sat, 9 Apr 2022 18:25:54 GMT, Vladimir Kozlov wrote:
> Testing passed.
Thank you Vladimir!
-
PR: https://git.openjdk.java.net/jdk/pull/7572
On Fri, 8 Apr 2022 00:55:50 GMT, Srinivas Vamsi Parasa
wrote:
>> I have few comments.
>
> Hi Vladimir (@vnkozlov),
>
> Incorporated all the suggestions you made in the previous review and pushed a
> new commit.
> Please let me know if anything else is needed.
>
which fuses
> division and modulus operations if needed. The DivMod optimization shows 3x
> improvement for Integer and ~65% improvement for Long.
Srinivas Vamsi Parasa has updated the pull request incrementally with one
additional commit since the last revision:
Fix the divmod crash due
which fuses
> division and modulus operations if needed. The DivMod optimization shows 3x
> improvement for Integer and ~65% improvement for Long.
Srinivas Vamsi Parasa has updated the pull request incrementally with one
additional commit since the last revision:
uncomment zero in int
On Wed, 6 Apr 2022 00:45:37 GMT, Vladimir Kozlov wrote:
>> Thanks for suggesting the enhancement. This enhancement will be implemented
>> as a part of https://bugs.openjdk.java.net/browse/JDK-8282365
>
> You do need `Ideal()` methods at least to check for dead code.
Added the Ideal() methods
On Wed, 6 Apr 2022 00:46:01 GMT, Vladimir Kozlov wrote:
>> Srinivas Vamsi Parasa has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> add error msg for jtreg test
>
> I have few comments.
Hi Vladimir
which fuses
> division and modulus operations if needed. The DivMod optimization shows 3x
> improvement for Integer and ~65% improvement for Long.
Srinivas Vamsi Parasa has updated the pull request incrementally with one
additional commit since the last revision:
Idead Ideal for udiv, umo
which fuses
> division and modulus operations if needed. The DivMod optimization shows 3x
> improvement for Integer and ~65% improvement for Long.
Srinivas Vamsi Parasa has updated the pull request with a new target base due
to a merge or a rebase. The pull request now contains 15 commits:
12 matches
Mail list logo