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

Reply via email to