RFR: 8291654: AArch64: assert from JDK-8287393 causes crashes

Evgeny Astigeevich duke at openjdk.org
Mon Aug 1 22:59:56 UTC 2022


On Mon, 1 Aug 2022 22:23:19 GMT, David Holmes <dholmes at openjdk.org> wrote:

> I can approve this as a trivial fix as it certainly cannot do any harm to remove the assertion and fix our CI. But it would be good if the reviewers of 8287393 chimed in at some point to confirm

It looks like the assert has UB in case of C1:

  static Compile* current() {
    return (Compile*) ciEnv::current()->compiler_data();
  }


In case of C1  `ciEnv::current()->compiler_data()` returns C1 `Compilation` and the cast to C2 `Compile` is invalid.

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

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


More information about the hotspot-compiler-dev mailing list