RFR: 8278104: C1 should support the compiler directive 'BreakAtExecute'

Guoxiong Li gli at openjdk.java.net
Sat Dec 18 04:19:25 UTC 2021


On Fri, 17 Dec 2021 19:41:45 GMT, Paul Hohensee <phh at openjdk.org> wrote:

>> Hi all,
>> 
>> Currently, the directive `BreakAtExecute` is not effective at C1. And the `CompileCommand=break` doesn't break the compiled method, too. This patch unifies the `BreakAtExecute` and `CompileCommand=break` to the directive 'BreakAtExecute' and uses the directive 'BreakAtExecute' to identify whether a breakpoint should be added.
>> 
>> The test group `hotspot_compiler` passed locally.(linux x86_64 fastdebug)
>> And the pre-submit tests passed before submitting the PR.
>> 
>> Thanks for taking the time to review.
>> 
>> Best Regards,
>> -- Guoxiong
>
> Lgtm.

@phohensee @vnkozlov Thanks for the review.

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

PR: https://git.openjdk.java.net/jdk/pull/6807


More information about the hotspot-compiler-dev mailing list