JDK-8255216 : Change _directive->BreakAtCompileOption to env()->break_at_compile()
  • Type: Enhancement
  • Component: hotspot
  • Sub-Component: compiler
  • Affected Version: 16
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • OS: generic
  • CPU: generic
  • Submitted: 2020-10-22
  • Updated: 2021-01-28
  • Resolved: 2021-01-22
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 17
17 b07Fixed
Related Reports
Relates :  
Description
ciEnv::_break_at_compile is set in compilerBroker but never be referred.
c1 and c2 check _directive->BreakAtCompileOption individually. 

it actually voids the following condtion.
  should_break = directive->BreakAtExecuteOption || task->check_break_at_flags();

first of all,  it sounds like it should be BreakAtCompileOption. 
second, the following flags are all ignored.

// CompileTask::check_break_at_flags
bool CompileTask::check_break_at_flags() {
  int compile_id = this->_compile_id;
  bool is_osr = (_osr_bci != CompileBroker::standard_entry_bci);

  if (CICountOSR && is_osr && (compile_id == CIBreakAtOSR)) {
    return true;
  } else {
    return (compile_id == CIBreakAt);
  }
} 
Comments
Changeset: 7be9113b Author: Dan Lemmond <dlemmond@amazon.com> Committer: Paul Hohensee <phh@openjdk.org> Date: 2021-01-22 20:33:44 +0000 URL: https://git.openjdk.java.net/jdk/commit/7be9113b
22-01-2021

Looks like the Compiler Control changes in JDK 9 made this obsolete (JDK-8137167).
23-10-2020