| commit | ce496cbda508bc648741d7da9785dce93a7a5f26 | [log] [tgz] |
|---|---|---|
| author | Xin Liu <xliu@openjdk.org> | Tue Dec 01 23:50:53 2020 +0000 |
| committer | Jie Fu <jiefu@openjdk.org> | Tue Dec 01 23:50:53 2020 +0000 |
| tree | f2711df0a87033885bd9e82e04f7d02fe0e23610 | |
| parent | 927504e8270b4ea44c10d7faf0959b107281ddfe [diff] |
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
For build instructions please see the online documentation, or either of these files:
See https://openjdk.java.net/ for more information about the OpenJDK Community and the JDK.