>> *) Is the move of ShenandoahHeapRegionSetIterator::ShenandoahHeapRegionSetIterator from .inline.hpp >> to .cpp sensible? This desyncs backports... > > It refused to compile without that change. Alternative seems to be to > mark the constructor volatile to avoid multiple definitions. inline not volatile. brain is already pudding. Roman