Skip to content

Commit

Permalink
Completely removed assert because it triggers bugs in mainline code
Browse files Browse the repository at this point in the history
  • Loading branch information
TobiHartmann committed Feb 23, 2021
1 parent 2a0faca commit bca88c7
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions src/hotspot/share/opto/library_call.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -109,9 +109,7 @@ JVMState* LibraryIntrinsic::generate(JVMState* jvms) {
#endif
ciMethod* callee = kit.callee();
const int bci = kit.bci();
#ifdef ASSERT
Node* ctrl = kit.control();
#endif

// Try to inline the intrinsic.
if ((CheckIntrinsics ? callee->intrinsic_candidate() : true) &&
kit.try_to_inline(_last_predicate)) {
Expand All @@ -133,8 +131,8 @@ JVMState* LibraryIntrinsic::generate(JVMState* jvms) {
C->print_inlining_update(this);
return kit.transfer_exceptions_into_jvms();
}

// The intrinsic bailed out
assert(ctrl == kit.control(), "Control flow was added although we bailed out");
if (jvms->has_method()) {
// Not a root compile.
const char* msg;
Expand Down

0 comments on commit bca88c7

Please sign in to comment.