8256843: [PPC64] runtime/logging/RedefineClasses.java fails with assert: registers not saved on stack #24
Please help review this small PPC64 fix.
Furthermore the fix changes os::current_frame() to return the correct
make run-test TEST=test/hotspot/jtreg/runtime/NMT/CheckForProperDetailStackTrace.java
Furthermore the inline assembler from os::current_stack_pointer() is
The fix passed our CI testing: JCK and JTREG, also in Xcomp mode, SPECjvm2008, SPECjbb2015, Renaissance Suite,
The text was updated successfully, but these errors were encountered:
@reinrich This change now passes all automated pre-integration checks.
After integration, the commit message for the final commit will be:
At the time when this comment was updated there had been 15 new commits pushed to the
As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid this automatic rebasing, please check the documentation for the /integrate command for further details.
Thanks for the review!
Yeah this would be nice but then we would be dependent to call inlining by the C++ compiler, i.e. we'd have to fiddle around with NMT_NOINLINE again (btw: we could do the same on s390 where NMT_NOINLINE is used)
@reinrich Since your change was applied there have been 27 commits pushed to the
Your commit was automatically rebased without conflicts.
Pushed as commit 1ce2e94.