[jdk8u-dev] RFR: 8303466: C2: failed: malformed control flow. Limit type made precise with MaxL/MinL
Martin Balao
mbalao at openjdk.org
Fri Jun 28 16:33:42 UTC 2024
Hi,
I would like to propose a backport of 8303466 [1] to jdk8u. jdk8u can benefit from this fix and having more accurate limit type information.
This pull request contains a backport of commit [8578e12c423ed61618e0b3ef81e5be3d18be1da2](https://github.com/openjdk/jdk11u-dev/commit/8578e12c423ed61618e0b3ef81e5be3d18be1da2) from the [openjdk/jdk11u-dev](https://github.com/openjdk/jdk11u-dev) repository.
This backport depends on [JDK-8262017](https://bugs.openjdk.org/browse/JDK-8262017).
The jdk11u patch does not apply cleanly because of the following:
* hotspot/src/share/vm/opto/addnode.hpp
* 8u does not have 8212043. Manually applied the change.
* hotspot/src/share/vm/runtime/vmStructs.cpp
* 8u does not have 8212043. Manually applied the change.
* hotspot/src/share/vm/opto/macro.cpp
* 8u does not have 8204210. Manually applied the change.
* 8u does not have 8186027. Manually applied the change.
* hotspot/src/share/vm/opto/loopTransform.cpp
* 8u does not have 8223142. In addition, 8u does not have 8034812. Manually applied the change.
* 8u does not have 8182299. Manually applied the change.
In addition, the following changes were made:
* Nodes allocation follows the pattern "new (C) ..." instead of "new ...".
* File locations were adjusted.
* In addnode.cpp, 8u does not have 8284358 so the function "Node::set_req_X(uint i, Node *n, PhaseGVN *gvn)" is not available. Picked this function (declaration and definition) from 8284358.
* In 8u, the jtreg VM prop vm.compiler2.enabled does not exist. C2 will likely be available and if the test were executed without C2, it should be harmless.
No regressions observed in hotspot:tier1.
Thanks,
Martin.-
--
[1] - https://bugs.openjdk.org/browse/JDK-8303466
-------------
Commit messages:
- Backport cc894d849aa5f730d5a806acfc7a237cf5170af1
Changes: https://git.openjdk.org/jdk8u-dev/pull/529/files
Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=529&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8303466
Stats: 392 lines in 11 files changed: 329 ins; 49 del; 14 mod
Patch: https://git.openjdk.org/jdk8u-dev/pull/529.diff
Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/529/head:pull/529
PR: https://git.openjdk.org/jdk8u-dev/pull/529
More information about the jdk8u-dev
mailing list