Some minor cleanup to G1SurvivorRegions and its uses should be done:

- make the regions array a direct member of the class (avoids potential memory leak, reduces one level of indirection)

- use the C++ for-each construct for loops over the regions array