Skip to content

Commit

Permalink
8304403: Remove unused methods in RangeCheckElimination::Bound
Browse files Browse the repository at this point in the history
Reviewed-by: chagedorn, thartmann
  • Loading branch information
enothum authored and TobiHartmann committed Jun 13, 2023
1 parent 9b0baa1 commit 6d05360
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 23 deletions.
20 changes: 0 additions & 20 deletions src/hotspot/share/c1/c1_RangeCheckElimination.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1337,26 +1337,6 @@ RangeCheckEliminator::Bound::Bound(Instruction::Condition cond, Value v, int con
}
}

// Set lower
void RangeCheckEliminator::Bound::set_lower(int value, Value v) {
assert(!v || !v->as_Constant() || !v->type()->as_IntConstant(), "Must not be constant!");
this->_lower = value;
this->_lower_instr = v;
}

// Set upper
void RangeCheckEliminator::Bound::set_upper(int value, Value v) {
assert(!v || !v->as_Constant() || !v->type()->as_IntConstant(), "Must not be constant!");
this->_upper = value;
this->_upper_instr = v;
}

// Add constant -> no overflow may occur
void RangeCheckEliminator::Bound::add_constant(int value) {
this->_lower += value;
this->_upper += value;
}

// or
void RangeCheckEliminator::Bound::or_op(Bound *b) {
// Watch out, bound is not guaranteed not to overflow!
Expand Down
3 changes: 0 additions & 3 deletions src/hotspot/share/c1/c1_RangeCheckElimination.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -97,12 +97,9 @@ class RangeCheckEliminator {
void and_op(Bound *b);
bool has_upper();
bool has_lower();
void set_upper(int upper, Value upper_instr);
void set_lower(int lower, Value lower_instr);
bool is_smaller(Bound *b);
void remove_upper();
void remove_lower();
void add_constant(int value);
Bound *copy();
};

Expand Down

1 comment on commit 6d05360

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.