RFA: 8168996: C2 crash at postaloc.cpp:140 : assert(false) failed: unexpected yanked node
Yangfei (Felix)
felix.yang at huawei.com
Mon Jan 11 12:37:42 UTC 2021
Hi Andrew,
>
> Hi Felix,
>
> On 11/01/2021 06:29, Yangfei (Felix) wrote:
> > Sorry, I should have mentioned more details about the diagnosis for 8u
> > in my previous mail.
> > . . .
> > With that we managed to get the C2 graph after "Optimized finished"
> > phase. The subgraph looks like:
> > LoadN --> DecodeN --> MemBarAcquire Here, the MemBarAcquire node
> > from a volatile object field was keeping the otherwise dead LoadN
> > nodes alive. But we don't have a Phi in between. This is slight
> > different with the C2 graph in the original RFR thread. I think this
> > does not affect the application of the original fix. Is this enough
> > to validate the 8u backport?
>
> That's more than enough to approve this as the right fix for the right issue!
>
> Nice work ;-)
Great to hear that resolves your concern :-) Thanks for reviewing and approving this.
Then I think I need push approval from the 8u maintainers. Anyone please?
Best Regards,
Felix
More information about the jdk8u-dev
mailing list