[lworld+vector] RFR: 8317699: [lworld+vector] Fix Vector API tests crash with "assert(vbox->is_Phi()) failed: should be phi"

Jatin Bhateja jbhateja at openjdk.org
Wed Oct 11 02:44:27 UTC 2023


On Wed, 11 Oct 2023 01:45:57 GMT, Xiaohong Gong <xgong at openjdk.org> wrote:

> > You mentioned about this being a problem in mainline also, can you kindly share a test reproducer for it, lworld+vector does diverg from mainline w.r.t VectorBoxNode which are now InlineTypeNode but VectorBoxAllocate is still the same. Pushing InlineTypeNode across phi is also done using _merge_through_phi_ routines.
> 
> Sure, I planed to add the test on jdk mainline once I push this change on that branch.

We do not see assertion failures in Vector API JTREG tests currently, which means above mentioned graph shape involving VectorBoxAllocate IR node only appears with lworld+vector for same test point. Was curious to know the root cause of this.

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

PR Comment: https://git.openjdk.org/valhalla/pull/937#issuecomment-1756660548



More information about the valhalla-dev mailing list