On Wed, 13 May 2026 23:33:08 GMT, Dean Long <[email protected]> wrote:
>> Marc Chevalier has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Specifying the dependency type > > I'm sure it's unrelated, but do you know if all the MemBarStoreStore nodes > are necessary? The way the graph is pruned it looks like there are 4 in a > row. @dean-long No, I don't know I fear. Also, I display only a fraction of the graph. Maybe the look less redundant with the whole graph. I've checked some of them, and they seem to be legitimately added, but maybe some could be removed later as optimizations go. Also, the state in the screenshot is just in the middle some IGVN, before loop opts, things can still happen. I can't check very easily, I don't have the graph opened just there. ------------- PR Comment: https://git.openjdk.org/valhalla/pull/2432#issuecomment-4458042747
