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