On Wed, 18 May 2022 15:44:10 GMT, Quan Anh Mai <d...@openjdk.java.net> wrote:
> This patch backs out the changes made by > [JDK-8285390](https://bugs.openjdk.java.net/browse/JDK-8285390) and > [JDK-8284742](https://bugs.openjdk.java.net/browse/JDK-8284742) since there > are failures due to div nodes floating above their validity checks. > > Thanks. Backout looks good except that you did not remove the test `test/hotspot/jtreg/compiler/integerArithmetic/TestDivision.java` which you've added here: https://github.com/openjdk/jdk/commit/b4a85cdae14eee895a0de2f26a2ffdd62b72bebc#diff-4daba70b215edefe9fd70b24f1e1052a51effdd48a1809e2ce5a06bc21a755c5 It's just a correctness test, so it would be okay to leave it in. But I guess it's better to backout everything. ------------- Changes requested by chagedorn (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/8774