Post-merge bug: Derby fails with assert(p != __null) failed: named projection 2 not found

Aleksey Shipilev shade at redhat.com
Thu Feb 15 16:12:46 UTC 2018


On 02/15/2018 05:09 PM, Roland Westrelin wrote:
> Here is the fix for that:
> 
> http://cr.openjdk.java.net/~roland/shenandoah/find_bottom_mem/webrev.00/

Looks good to me.

Two questions:

 1) Is this backportable? I.e. should the same fix go to sh/jdk9 and sh/jdk9?

 2) I see two other mentions of OptoRuntime::rethrow_stub in asserts, should new entry points be
added there as well?

        assert(u->adr_type() != NULL ||
               u->Opcode() == Op_Rethrow ||
               u->Opcode() == Op_Return ||
               u->Opcode() == Op_SafePoint ||
               (u->is_CallStaticJava() && u->as_CallStaticJava()->uncommon_trap_request() != 0) ||
               (u->is_CallStaticJava() && u->as_CallStaticJava()->_entry_point ==
OptoRuntime::rethrow_stub()) ||
               u->Opcode() == Op_CallLeaf, "");

Thanks,
-Aleksey



More information about the shenandoah-dev mailing list