RFR: 8348212: Need to add warn() step to JavacTaskImpl after JDK-8344148 [v3]

Archie Cobbs acobbs at openjdk.org
Fri Jan 24 17:24:08 UTC 2025


> In [JDK-8344148](https://bugs.openjdk.org/browse/JDK-8344148) a new `warn()` compiler phase was added and `JavaCompiler.java` was updated accordingly.
> 
> However, the class `JavacTaskImpl` also walks through the compiler phases step-by-step, but the new `warn()` step was never added there. This will cause some warnings to not be emitted when that API is used.
> 
> This PR adds the missing `warn()` calls.

Archie Cobbs has updated the pull request incrementally with one additional commit since the last revision:

  Avoid having the regression test class extend another regression test class.

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/23223/files
  - new: https://git.openjdk.org/jdk/pull/23223/files/c7b6fcb5..a9b5f2d6

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=23223&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=23223&range=01-02

  Stats: 12 lines in 1 file changed: 6 ins; 3 del; 3 mod
  Patch: https://git.openjdk.org/jdk/pull/23223.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/23223/head:pull/23223

PR: https://git.openjdk.org/jdk/pull/23223


More information about the compiler-dev mailing list