[lworld] Integrated: 8262308: [lworld] Various small C2 fixes for bugs found by stress testing
thartmann at openjdk.java.net
Thu Feb 25 11:04:56 UTC 2021
On Thu, 25 Feb 2021 07:31:27 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:
> More testing with stress flags enabled revealed the following bugs:
> - We should not call `remove_dead_node` from `InlineTypeBaseNode::Ideal`. Instead, we should just make sure the dead node is on the IGVN worklist to be removed. I've also removed the `VerifyVM` option from the framework. Such flags can be added manually or via the CI job definitions.
> - `PhaseMacroExpand::process_users_of_allocation` needs to handle `InlineTypePtrNodes` because we now keep them until after macro expansion.
> - `GraphKit::type_check_receiver` should create a `InlineTypePtrNode` instead of a `InlineTypeNode` because it operates on pointers.
> I have a job definition file with all these stress flags enabled that I will integrate separately soon.
> Best regards,
This pull request has now been integrated.
Author: Tobias Hartmann <thartmann at openjdk.org>
Stats: 21 lines in 4 files changed: 11 ins; 7 del; 3 mod
8262308: [lworld] Various small C2 fixes for bugs found by stress testing
More information about the valhalla-dev