RFR: JDK-8278329: some TraceDeoptimization code not included in PRODUCT build
Tobias Holenstein
duke at openjdk.java.net
Wed Dec 15 12:21:17 UTC 2021
After "JDK-8154011: Make `TraceDeoptimization` a diagnostic flag" some code was not included in the PRODUCT build.
Removed all the #ifndef PRODUCT guards around `TraceDeoptimization` checks and made sure to be consistent.
The DEOPT PACKING messages were controlled by `PrintDeoptimizationDetails` (develop flag), but DEOPT UNPACKING is controlled by `TraceDeoptimization` (product flag),. Therefore changed DEOPT PACKING messages to be controlled by `TraceDeoptimization` as well.
Checked that tests are not affected. Checked on Aurora that performance is not affected.
-------------
Commit messages:
- missing #ifndef PRODUCT added
- restore PrintDeoptimizationDetails as develop flag
- remove UnlockDiagnosticVMOptions duplicate in TestDeoptOOM.java
- Update TestDeoptOMM.cpp flags
- UnlockDiagnosticVMOptions in TestDeoptOOM.java
- Merge remote-tracking branch 'origin/master' into JDK-8278329
- fixed some formatting of PrintDeoptimizationDetails and TraceDeoptimization
- JDK-8278329: some TraceDeoptimization code not included in PRODUCT build
Changes: https://git.openjdk.java.net/jdk/pull/6746/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=6746&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8278329
Stats: 118 lines in 4 files changed: 45 ins; 54 del; 19 mod
Patch: https://git.openjdk.java.net/jdk/pull/6746.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/6746/head:pull/6746
PR: https://git.openjdk.java.net/jdk/pull/6746
More information about the hotspot-dev
mailing list