Skip to content
This repository has been archived by the owner. It is now read-only.
Permalink
Browse files
8258937: Remove JVM IgnoreRewrites flag
Reviewed-by: coleenp
  • Loading branch information
Harold Seigel committed Jan 5, 2021
1 parent 4d3d599 commit cf9908b437cb3847c1e813f6eccdc1d3326742ff
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 29 deletions.
@@ -1,5 +1,5 @@
/*
* Copyright (c) 1997, 2020, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 1997, 2021, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -2263,34 +2263,11 @@ void GenerateOopMap::rewrite_refval_conflicts()
return;

// Check if rewrites are allowed in this parse.
if (!allow_rewrites() && !IgnoreRewrites) {
if (!allow_rewrites()) {
fatal("Rewriting method not allowed at this stage");
}


// This following flag is to tempoary supress rewrites. The locals that might conflict will
// all be set to contain values. This is UNSAFE - however, until the rewriting has been completely
// tested it is nice to have.
if (IgnoreRewrites) {
if (Verbose) {
tty->print("rewrites suppressed for local no. ");
for (int l = 0; l < _max_locals; l++) {
if (_new_var_map[l] != l) {
tty->print("%d ", l);
vars()[l] = CellTypeState::value;
}
}
tty->cr();
}

// That was that...
_new_var_map = NULL;
_nof_refval_conflicts = 0;
_conflict = false;

return;
}

// Tracing flag
_did_rewriting = true;

@@ -674,10 +674,6 @@ const intx ObjectAlignmentInBytes = 8;
"Tell whether the VM should register soft/weak/final/phantom " \
"references") \
\
develop(bool, IgnoreRewrites, false, \
"Suppress rewrites of bytecodes in the oopmap generator. " \
"This is unsafe!") \
\
develop(bool, PrintCodeCacheExtension, false, \
"Print extension of code cache") \
\

0 comments on commit cf9908b

Please sign in to comment.