Skip to content

8348186: C1: Purge fpu_stack_size infrastructure #23215

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

8348186: C1: Purge fpu_stack_size infrastructure #23215

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion src/hotspot/cpu/x86/c1_LinearScan_x86.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -649,7 +649,6 @@ void FpuStackAllocator::handle_op1(LIR_Op1* op1) {
new_in = to_fpu_stack_top(res);
new_res = new_in;

op1->set_fpu_stack_size(sim()->stack_size());
break;
}

Expand Down
9 changes: 0 additions & 9 deletions src/hotspot/share/c1/c1_LIR.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1599,8 +1599,6 @@ class LIR_OpTypeCheck: public LIR_Op {
class LIR_Op2: public LIR_Op {
friend class LIR_OpVisitState;

int _fpu_stack_size; // for sin/cos implementation on Intel

protected:
LIR_Opr _opr1;
LIR_Opr _opr2;
Expand All @@ -1617,7 +1615,6 @@ class LIR_Op2: public LIR_Op {
public:
LIR_Op2(LIR_Code code, LIR_Condition condition, LIR_Opr opr1, LIR_Opr opr2, CodeEmitInfo* info = nullptr, BasicType type = T_ILLEGAL)
: LIR_Op(code, LIR_OprFact::illegalOpr, info)
, _fpu_stack_size(0)
, _opr1(opr1)
, _opr2(opr2)
, _tmp1(LIR_OprFact::illegalOpr)
Expand All @@ -1632,7 +1629,6 @@ class LIR_Op2: public LIR_Op {

LIR_Op2(LIR_Code code, LIR_Condition condition, LIR_Opr opr1, LIR_Opr opr2, LIR_Opr result, BasicType type)
: LIR_Op(code, result, nullptr)
, _fpu_stack_size(0)
, _opr1(opr1)
, _opr2(opr2)
, _tmp1(LIR_OprFact::illegalOpr)
Expand All @@ -1649,7 +1645,6 @@ class LIR_Op2: public LIR_Op {
LIR_Op2(LIR_Code code, LIR_Opr opr1, LIR_Opr opr2, LIR_Opr result = LIR_OprFact::illegalOpr,
CodeEmitInfo* info = nullptr, BasicType type = T_ILLEGAL)
: LIR_Op(code, result, info)
, _fpu_stack_size(0)
, _opr1(opr1)
, _opr2(opr2)
, _tmp1(LIR_OprFact::illegalOpr)
Expand All @@ -1665,7 +1660,6 @@ class LIR_Op2: public LIR_Op {
LIR_Op2(LIR_Code code, LIR_Opr opr1, LIR_Opr opr2, LIR_Opr result, LIR_Opr tmp1, LIR_Opr tmp2 = LIR_OprFact::illegalOpr,
LIR_Opr tmp3 = LIR_OprFact::illegalOpr, LIR_Opr tmp4 = LIR_OprFact::illegalOpr, LIR_Opr tmp5 = LIR_OprFact::illegalOpr)
: LIR_Op(code, result, nullptr)
, _fpu_stack_size(0)
, _opr1(opr1)
, _opr2(opr2)
, _tmp1(tmp1)
Expand Down Expand Up @@ -1693,9 +1687,6 @@ class LIR_Op2: public LIR_Op {
assert(code() == lir_cmp || code() == lir_branch || code() == lir_cond_float_branch, "only valid for branch"); _condition = condition;
}

void set_fpu_stack_size(int size) { _fpu_stack_size = size; }
int fpu_stack_size() const { return _fpu_stack_size; }

void set_in_opr1(LIR_Opr opr) { _opr1 = opr; }
void set_in_opr2(LIR_Opr opr) { _opr2 = opr; }

Expand Down