Tobias Hartmann comments

Results 27 comments of


                                            Tobias Hartmann

8329538: Accelerate P256 on x86_64 using Montgomery intrinsic

Unfortunately, this caused a performance regression, see [JDK-8333583](https://bugs.openjdk.org/browse/JDK-8333583). @vpaprotsk, please have a look.

8327381: Refactor type-improving transformations in BoolNode::Ideal to BoolNode::Value

@tabjy any update on this? Let us know if you need help with reproducing the issue that Christian reported. Thanks!

8327381: Refactor type-improving transformations in BoolNode::Ideal to BoolNode::Value

Thanks @tabjy! @chhagedorn, our IR Framework expert, is currently on vacation but will be back later this week.

8328528: C2 should optimize long-typed parallel iv in an int counted loop

`compiler/loopopts/parallel_iv/TestParallelIvInIntCountedLoop.java` times out in our testing both with `-XX:StressLongCountedLoop=200000000` and with `-XX:+UnlockExperimentalVMOptions -XX:PerMethodSpecTrapLimit=0 -XX:PerMethodTrapLimit=0`: ``` "main" #1 [2771172] prio=5 os_prio=0 cpu=500187.70ms elapsed=503.08s allocated=6554K defined_classes=227 tid=0x0000ffff9002d550 nid=2771172 runnable [0x0000ffff972bf000] java.lang.Thread.State: RUNNABLE...

8328528: C2 should optimize long-typed parallel iv in an int counted loop

Thanks for the detailed investigation and feedback. The changes look good to me, I'll re-run testing and report back.

8328528: C2 should optimize long-typed parallel iv in an int counted loop

@tabjy We are still seeing timeouts with `-XX:+UnlockDiagnosticVMOptions -XX:TieredStopAtLevel=3 -XX:+StressLoopInvariantCodeMotion -XX:+StressRangeCheckElimination -XX:+StressLinearScan`. Maybe the test should be enabled only if C2 is available.

8319822: Use a linear-time algorithm for assert_different_registers()

The build fails on Windows: ``` [2023-11-13T08:04:09,465Z] c:\sb\prod\1699862498\workspace\open\src\hotspot\cpu\x86\register_x86.hpp(402): error C2220: the following warning is treated as an error [2023-11-13T08:04:09,465Z] c:\sb\prod\1699862498\workspace\open\src\hotspot\cpu\x86\register_x86.hpp(402): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss...