RFR: 8344148: Add an explicit compiler phase for warning generation [v3]
Vicente Romero
vromero at openjdk.org
Mon Jan 6 18:18:37 UTC 2025
On Thu, 2 Jan 2025 19:17:54 GMT, Archie Cobbs <acobbs at openjdk.org> wrote:
>> Please review this patch which does some minor refactoring to the compiler:
>> * Create a new `WARN` phase which can be a dedicated home for (new) lint/warning logic
>> * Create a new `WarningAnalyzer` singleton whose job is to invoke such lint/warning logic
>> * Move `ThisEscapeAnalyzer` out of `Flow` (where it doesn't belong) and into `WarningAnalyzer`
>> * Refactor `ThisEscapeAnalyzer` to be a context singleton like all other such classes
>>
>> See [JDK-8344148](https://bugs.openjdk.org/browse/JDK-8344148) for details.
>
> Archie Cobbs has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:
>
> - Bump copyright year to 2025.
> - Merge branch 'master' into JDK-8344148
> - Merge branch 'master' into JDK-8344148
> - Merge branch 'master' into JDK-8344148
> - Update copyright years.
> - Add an explicit compiler phase for warning generation.
lgtm
-------------
Marked as reviewed by vromero (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/22088#pullrequestreview-2532708559
More information about the compiler-dev
mailing list