Commit 545c04b9 authored by yangguo@chromium.org's avatar yangguo@chromium.org

Minor-key-ify BinaryOpICWithAllocationSiteStub.

R=mvstanton@chromium.org

Review URL: https://codereview.chromium.org/506003002

git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@23444 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
parent af08d3f2
...@@ -3723,7 +3723,7 @@ void BinaryOpICWithAllocationSiteStub::Generate(MacroAssembler* masm) { ...@@ -3723,7 +3723,7 @@ void BinaryOpICWithAllocationSiteStub::Generate(MacroAssembler* masm) {
// Tail call into the stub that handles binary operations with allocation // Tail call into the stub that handles binary operations with allocation
// sites. // sites.
BinaryOpWithAllocationSiteStub stub(isolate(), state_); BinaryOpWithAllocationSiteStub stub(isolate(), state());
__ TailCallStub(&stub); __ TailCallStub(&stub);
} }
......
...@@ -4332,7 +4332,7 @@ void BinaryOpICWithAllocationSiteStub::Generate(MacroAssembler* masm) { ...@@ -4332,7 +4332,7 @@ void BinaryOpICWithAllocationSiteStub::Generate(MacroAssembler* masm) {
// Tail call into the stub that handles binary operations with allocation // Tail call into the stub that handles binary operations with allocation
// sites. // sites.
BinaryOpWithAllocationSiteStub stub(isolate(), state_); BinaryOpWithAllocationSiteStub stub(isolate(), state());
__ TailCallStub(&stub); __ TailCallStub(&stub);
} }
......
...@@ -291,7 +291,7 @@ void BinaryOpICWithAllocationSiteStub::GenerateAheadOfTime(Isolate* isolate) { ...@@ -291,7 +291,7 @@ void BinaryOpICWithAllocationSiteStub::GenerateAheadOfTime(Isolate* isolate) {
void BinaryOpICWithAllocationSiteStub::PrintState( void BinaryOpICWithAllocationSiteStub::PrintState(
OStream& os) const { // NOLINT OStream& os) const { // NOLINT
os << state_; os << state();
} }
......
...@@ -1201,7 +1201,7 @@ class BinaryOpICWithAllocationSiteStub V8_FINAL : public PlatformCodeStub { ...@@ -1201,7 +1201,7 @@ class BinaryOpICWithAllocationSiteStub V8_FINAL : public PlatformCodeStub {
public: public:
BinaryOpICWithAllocationSiteStub(Isolate* isolate, BinaryOpICWithAllocationSiteStub(Isolate* isolate,
const BinaryOpIC::State& state) const BinaryOpIC::State& state)
: PlatformCodeStub(isolate), state_(state) { : PlatformCodeStub(isolate) {
minor_key_ = state.GetExtraICState(); minor_key_ = state.GetExtraICState();
} }
...@@ -1218,11 +1218,11 @@ class BinaryOpICWithAllocationSiteStub V8_FINAL : public PlatformCodeStub { ...@@ -1218,11 +1218,11 @@ class BinaryOpICWithAllocationSiteStub V8_FINAL : public PlatformCodeStub {
} }
virtual InlineCacheState GetICState() const V8_OVERRIDE { virtual InlineCacheState GetICState() const V8_OVERRIDE {
return state_.GetICState(); return state().GetICState();
} }
virtual ExtraICState GetExtraICState() const V8_OVERRIDE { virtual ExtraICState GetExtraICState() const V8_OVERRIDE {
return state_.GetExtraICState(); return static_cast<ExtraICState>(minor_key_);
} }
virtual void Generate(MacroAssembler* masm) V8_OVERRIDE; virtual void Generate(MacroAssembler* masm) V8_OVERRIDE;
...@@ -1232,14 +1232,15 @@ class BinaryOpICWithAllocationSiteStub V8_FINAL : public PlatformCodeStub { ...@@ -1232,14 +1232,15 @@ class BinaryOpICWithAllocationSiteStub V8_FINAL : public PlatformCodeStub {
virtual Major MajorKey() const V8_OVERRIDE { virtual Major MajorKey() const V8_OVERRIDE {
return BinaryOpICWithAllocationSite; return BinaryOpICWithAllocationSite;
} }
virtual uint32_t MinorKey() const V8_OVERRIDE { return GetExtraICState(); }
private: private:
BinaryOpIC::State state() const {
return BinaryOpIC::State(isolate(), static_cast<ExtraICState>(minor_key_));
}
static void GenerateAheadOfTime(Isolate* isolate, static void GenerateAheadOfTime(Isolate* isolate,
const BinaryOpIC::State& state); const BinaryOpIC::State& state);
BinaryOpIC::State state_;
DISALLOW_COPY_AND_ASSIGN(BinaryOpICWithAllocationSiteStub); DISALLOW_COPY_AND_ASSIGN(BinaryOpICWithAllocationSiteStub);
}; };
......
...@@ -3626,7 +3626,7 @@ void BinaryOpICWithAllocationSiteStub::Generate(MacroAssembler* masm) { ...@@ -3626,7 +3626,7 @@ void BinaryOpICWithAllocationSiteStub::Generate(MacroAssembler* masm) {
// Tail call into the stub that handles binary operations with allocation // Tail call into the stub that handles binary operations with allocation
// sites. // sites.
BinaryOpWithAllocationSiteStub stub(isolate(), state_); BinaryOpWithAllocationSiteStub stub(isolate(), state());
__ TailCallStub(&stub); __ TailCallStub(&stub);
} }
......
...@@ -3593,7 +3593,7 @@ void BinaryOpICWithAllocationSiteStub::Generate(MacroAssembler* masm) { ...@@ -3593,7 +3593,7 @@ void BinaryOpICWithAllocationSiteStub::Generate(MacroAssembler* masm) {
// Tail call into the stub that handles binary operations with allocation // Tail call into the stub that handles binary operations with allocation
// sites. // sites.
BinaryOpWithAllocationSiteStub stub(isolate(), state_); BinaryOpWithAllocationSiteStub stub(isolate(), state());
__ TailCallStub(&stub); __ TailCallStub(&stub);
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment