Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
8253040: Remove unused Matcher::regnum_to_fpu_offset()
Reviewed-by: adityam, vlivanov
  • Loading branch information
JohnTortugo authored and shipilev committed Sep 16, 2020
1 parent 7c564e1 commit fbf4699
Show file tree
Hide file tree
Showing 7 changed files with 0 additions and 33 deletions.
6 changes: 0 additions & 6 deletions src/hotspot/cpu/aarch64/aarch64.ad
Expand Up @@ -2410,12 +2410,6 @@ const int Matcher::float_pressure(int default_pressure_threshold) {
return default_pressure_threshold;
}

int Matcher::regnum_to_fpu_offset(int regnum)
{
Unimplemented();
return 0;
}

// Is this branch offset short enough that a short branch can be used?
//
// NOTE: If the platform does not provide any short branch variants, then
Expand Down
4 changes: 0 additions & 4 deletions src/hotspot/cpu/arm/arm.ad
Expand Up @@ -997,10 +997,6 @@ const int Matcher::float_pressure(int default_pressure_threshold) {
return default_pressure_threshold;
}

int Matcher::regnum_to_fpu_offset(int regnum) {
return regnum - 32; // The FP registers are in the second chunk
}

// Vector width in bytes
const int Matcher::vector_width_in_bytes(BasicType bt) {
return MaxVectorSize;
Expand Down
6 changes: 0 additions & 6 deletions src/hotspot/cpu/ppc/ppc.ad
Expand Up @@ -2165,12 +2165,6 @@ const int Matcher::float_pressure(int default_pressure_threshold) {
return default_pressure_threshold;
}

int Matcher::regnum_to_fpu_offset(int regnum) {
// No user for this method?
Unimplemented();
return 999;
}

const bool Matcher::convL2FSupported(void) {
// fcfids can do the conversion (>= Power7).
// fcfid + frsp showed rounding problem when result should be 0x3f800001.
Expand Down
5 changes: 0 additions & 5 deletions src/hotspot/cpu/s390/s390.ad
Expand Up @@ -1569,11 +1569,6 @@ const bool Matcher::match_rule_supported_vector(int opcode, int vlen, BasicType
return ret_value; // Per default match rules are supported.
}

int Matcher::regnum_to_fpu_offset(int regnum) {
ShouldNotReachHere();
return regnum - 32; // The FP registers are in the second chunk.
}

const bool Matcher::has_predicated_vectors(void) {
return false;
}
Expand Down
4 changes: 0 additions & 4 deletions src/hotspot/cpu/x86/x86_32.ad
Expand Up @@ -1371,10 +1371,6 @@ uint MachUEPNode::size(PhaseRegAlloc *ra_) const {

//=============================================================================

int Matcher::regnum_to_fpu_offset(int regnum) {
return regnum - 32; // The FP registers are in the second chunk
}

// This is UltraSparc specific, true just means we have fast l2f conversion
const bool Matcher::convL2FSupported(void) {
return true;
Expand Down
5 changes: 0 additions & 5 deletions src/hotspot/cpu/x86/x86_64.ad
Expand Up @@ -1578,11 +1578,6 @@ uint MachUEPNode::size(PhaseRegAlloc* ra_) const

//=============================================================================

int Matcher::regnum_to_fpu_offset(int regnum)
{
return regnum - 32; // The FP registers are in the second chunk
}

// This is UltraSparc specific, true just means we have fast l2f conversion
const bool Matcher::convL2FSupported(void) {
return true;
Expand Down
3 changes: 0 additions & 3 deletions src/hotspot/share/opto/matcher.hpp
Expand Up @@ -437,9 +437,6 @@ class Matcher : public PhaseTransform {
OptoReg::Name c_frame_pointer() const;
static RegMask c_frame_ptr_mask;

// !!!!! Special stuff for building ScopeDescs
virtual int regnum_to_fpu_offset(int regnum);

// Is this branch offset small enough to be addressed by a short branch?
bool is_short_branch_offset(int rule, int br_size, int offset);

Expand Down

1 comment on commit fbf4699

@bridgekeeper
Copy link

@bridgekeeper bridgekeeper bot commented on fbf4699 Sep 16, 2020

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.