On Wed, 23 Apr 2025 17:14:17 GMT, Aleksey Shipilev <sh...@openjdk.org> wrote:
> A lot of SA infrastructure was added to support SA compiler replay with > [JDK-7088955](https://bugs.openjdk.org/browse/JDK-7088955). With > [JDK-8315488](https://bugs.openjdk.org/browse/JDK-8315488), we got rid from > the most of it. `CompileTask` seems to be left behind. Nothing uses it in SA > now. > > Now, for Leyden, we want to massage `CompileTask` for better performance and > reliability ([JDK-8231269](https://bugs.openjdk.org/browse/JDK-8231269)), and > keeping `CompileTask` in SA would require us to implement a whole bunch of > complicated, but unnecessary code. > > So, it would be good to purge `CompileTask` from SA. > > Note that I left the related `vmStructs` definitions, because async-profiler > uses those; I think to see which methods current compiler is compiling. That > use looks safe, as it polls the task from the already set up ciEnv. > async-profiler would need to re-adjust after > [JDK-8231269](https://bugs.openjdk.org/browse/JDK-8231269) makes relevant > changes in `vmStructs`. This PR frees us from doing the same thing in SA. This pull request has now been integrated. Changeset: 0edd018a Author: Aleksey Shipilev <sh...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/0edd018a48c202a6da4afe80e245799b47000885 Stats: 73 lines in 1 file changed: 0 ins; 73 del; 0 mod 8355432: Remove CompileTask from SA Reviewed-by: cjplummer, lmesnik ------------- PR: https://git.openjdk.org/jdk/pull/24832
