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