RFR: 8350097: Make Compilation::current() and Compile::current() safer
Thomas Stuefe
stuefe at openjdk.org
Fri Feb 14 18:53:11 UTC 2025
On Fri, 14 Feb 2025 17:57:31 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:
>> Context: I think `ciEnv` is pretty compiler-agnostic, and it would be better to avoid exposing the fact C1/C2 exist to that interface, even if only for asserts. Seems cleaner to check that we are calling `Compilation::current()` from C1 and `Compile::current()` from C2.
>
> I agree with Aleksey here.
Yes, I like this too. Simpler. Okay, I try that.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23635#discussion_r1956588610
More information about the hotspot-compiler-dev
mailing list