[jdk16] RFR: 8258619: -XX:CompileCommand=blackhole, * should only accept empty static methods

Aleksey Shipilev shade at openjdk.java.net
Thu Dec 17 18:23:14 UTC 2020


In JDK-8257827 CSR, a further comment was raised about changing the semantics of the method. This allegedly introduces the conformance problem. While it is still an open question where the conformance problem is real, we can sidestep the entire issue by allowing only empty static methods to be blackholed. This eliminates any behavioral difference: empty methods have no side effects, and static methods do not need any null checks.

This change eliminates a lot of new code added by original JDK-8252505 push, mostly now-useless testing code. It adds two more tests that verify that non-empty and non-static methods are properly rejected.

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

Commit messages:
 - Amend assertion messages
 - 8258619: -XX:CompileCommand=blackhole,* should only accept empty static methods

Changes: https://git.openjdk.java.net/jdk16/pull/45/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk16&pr=45&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8258619
  Stats: 1180 lines in 11 files changed: 203 ins; 914 del; 63 mod
  Patch: https://git.openjdk.java.net/jdk16/pull/45.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk16 pull/45/head:pull/45

PR: https://git.openjdk.java.net/jdk16/pull/45


More information about the hotspot-compiler-dev mailing list