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

[Guoxiong Li]

On Sun, 12 Dec 2021 04:21:51 GMT, Guoxiong Li <gli 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

This pull request has now been integrated.

Changeset: 3c10b5db
Author:    Guoxiong Li <gli at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/3c10b5db38455b8aed88599f5743fd846bd0913e
Stats:     26 lines in 8 files changed: 12 ins; 0 del; 14 mod

8278104: C1 should support the compiler directive 'BreakAtExecute'

Reviewed-by: xliu, phh, kvn

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

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