git: openjdk/mobile: 2 new changesets
duke
duke at openjdk.java.net
Tue Dec 1 23:51:58 UTC 2020
Changeset: ce496cbd
Author: Xin Liu <xliu at openjdk.org>
Committer: Jie Fu <jiefu at openjdk.org>
Date: 2020-12-01 23:50:53 +0000
URL: https://git.openjdk.java.net/mobile/commit/ce496cbd
8257190: simplify PhaseIdealLoop constructors
Currently, C2 has 3 private constructors of PhaseIdealLoop as follows. a-b are for verification. only c is for real loop optimizations.
a. PhaseIdealLoop( PhaseIterGVN &igvn)
b. PhaseIdealLoop(PhaseIterGVN &igvn, const PhaseIdealLoop *verify_me)
c. PhaseIdealLoop(PhaseIterGVN &igvn, LoopOptsMode mode)
I propose 3 changes to simplify them.
1. add assertion in the constructor c. C2 shouldn't use mode = LoopOptsVerify for it.
2. merge a and b into one constructor.
3. make the merged verification ctor only for debug builds.
Reviewed-by: thartmann, kvn
! src/hotspot/share/opto/loopnode.hpp
Changeset: 2e845130
Author: duke <duke at openjdk.org>
Date: 2020-12-01 23:51:25 +0000
URL: https://git.openjdk.java.net/mobile/commit/2e845130
Automatic merge of jdk:master into master
More information about the mobile-changes
mailing list