Staffan, You was right, only two classes is C2 specific.
Here is updated webrev. http://cr.openjdk.java.net/~dsamersoff/JDK-8023667/webrev.02/ -Dmitry On 2014-02-03 16:13, Staffan Larsen wrote: > > On 3 feb 2014, at 11:59, Dmitry Samersoff <dmitry.samers...@oracle.com> wrote: > >> Staffan, >> >> All items under try-catch exists only in server (C2 compiler) VM, so if >> one of them is missing in target VM other items is missed as well. > > Are you sure? Some of them look very generic: “JavaThread”, > “SurrogateLockerThread”, “GenCollectedHeap”, etc. Perhaps the > catch-ignore-c2-specifics should end at lin 821? > > /Staffan > > >> >> I have no ideas what happens later if SA tries to access one of this >> non-initialized items, but I don't see any errors in my home testing. >> >> -Dmitry >> >> On 2014-02-03 14:39, Staffan Larsen wrote: >>> It looks like this new version will leave some things uninitialized >>> if an item higher on the list throws an exception. I don’t think that >>> is what you want. >>> >>> /Staffan >>> >>> >>> On 31 jan 2014, at 20:42, Dmitry Samersoff >>> <dmitry.samers...@oracle.com> wrote: >>> >>>> Hi Everybody, >>>> >>>> Please review the fix: >>>> >>>> http://cr.openjdk.java.net/~dsamersoff/JDK-8023667/webrev.01/ >>>> >>>> sa.js try to initialize all possible VM structures regardless of >>>> type of VM we are running on >>>> >>>> -- Dmitry Samersoff Oracle Java development team, Saint Petersburg, >>>> Russia * I would love to change the world, but they won't give me >>>> the sources. >>> >> >> >> -- >> Dmitry Samersoff >> Oracle Java development team, Saint Petersburg, Russia >> * I would love to change the world, but they won't give me the sources. > -- Dmitry Samersoff Oracle Java development team, Saint Petersburg, Russia * I would love to change the world, but they won't give me the sources.
