Skip to content

Commit

Permalink
No need to synchronize the instruction stream when not patching.
Browse files Browse the repository at this point in the history
Removing the use of 'maybe_isb()' in the patching stub since we are
effectively not patching the instruction stream.
  • Loading branch information
Patric Hedlin committed Sep 24, 2020
1 parent 06ab4a5 commit aab55c1
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/hotspot/cpu/aarch64/c1_Runtime1_aarch64.cpp
Expand Up @@ -555,7 +555,7 @@ OopMapSet* Runtime1::generate_patching(StubAssembler* sasm, address target) {
#endif

__ reset_last_Java_frame(true);
__ maybe_isb();

#ifdef ASSERT
// check that fields in JavaThread for exception oop and issuing pc are empty
Label oop_empty;
Expand All @@ -575,7 +575,7 @@ OopMapSet* Runtime1::generate_patching(StubAssembler* sasm, address target) {
// expected scenario and anything else is an error. Note that we maintain a
// check on the result purely as a defensive measure.
Label no_deopt;
__ cbz(r0, no_deopt); // have we deoptimized?
__ cbz(r0, no_deopt); // Have we deoptimized?

// Perform a re-execute. The proper return address is already on the stack,
// we just need to restore registers, pop all of our frame but the return
Expand Down

0 comments on commit aab55c1

Please sign in to comment.