RFR: 8301338: Identical branch conditions in CompileBroker::print_heapinfo

Aleksey Shipilev shade at openjdk.org
Mon Jan 30 10:28:58 UTC 2023


This was silently introduced by [JDK-8219586](https://bugs.openjdk.org/browse/JDK-8219586). SonarCloud complains:
 "Identical sub-expressions on both sides of operator "||".

There are two occurrences of:

 if ((function_lock_1 != NULL) || (function_lock_1 != NULL)) {


Additional testing:
 - [x] Linux x86_64 fastdebug, `-XX:+PrintCodeHeapAnalytics` adhoc run
 - [x] Linux x86_64 fastdebug, `compiler/codecache`

-------------

Commit messages:
 - Fix

Changes: https://git.openjdk.org/jdk/pull/12288/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=12288&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8301338
  Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/12288.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/12288/head:pull/12288

PR: https://git.openjdk.org/jdk/pull/12288


More information about the hotspot-compiler-dev mailing list