Prepared by: | twisti on Mon Oct 14 11:14:45 PDT 2013 |
---|---|
Workspace: | /Users/twisti/ws/8026376 |
Compare against: | /Users/twisti/ws/hotspot |
Summary of changes: | 2 lines changed: 2 ins; 0 del; 0 mod; 3925 unchg |
Patch of changes: | 8026376.patch |
Author comments: |
8026376: assert(false) failed: DEBUG MESSAGE: exception pc already set Reviewed-by: It made me suspicious that only the second assert fired (the exception pc check) and not the first one (the exception oop check). I checked all the places where JavaThread::exception_oop_offset() was used to clear the exception oop to see if we also clear the exception pc. In the deopt blob we have such a case. On x86 we clear both but on SPARC we don't. Added the missing null-store instruction makes the test case pass. Reading the exception pc is not really required but I've added it for completeness. |
Bug id: | 8026376 |
Legend: |
Modified file Deleted file New file |
Cdiffs
Udiffs
Sdiffs
Frames
Old
New
Patch
Raw
src/cpu/sparc/vm/sharedRuntime_sparc.cpp
2 lines changed: 2 ins; 0 del; 0 mod; 3925 unchg
This code review page was prepared using /Users/twisti/bin/webrev (vers 23.18-hg-never-dev).