Commit cfb957af authored by danno@chromium.org's avatar danno@chromium.org

Use IsRuntimeEntry for RUNTIME_ENTRY comparison

Review URL: https://codereview.chromium.org/12702005
Patch from Haitao Feng <haitao.feng@intel.com>.

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13929 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
parent 30ca31f4
...@@ -107,13 +107,13 @@ void RelocInfo::apply(intptr_t delta) { ...@@ -107,13 +107,13 @@ void RelocInfo::apply(intptr_t delta) {
Address RelocInfo::target_address() { Address RelocInfo::target_address() {
ASSERT(IsCodeTarget(rmode_) || rmode_ == RUNTIME_ENTRY); ASSERT(IsCodeTarget(rmode_) || IsRuntimeEntry(rmode_));
return Assembler::target_address_at(pc_); return Assembler::target_address_at(pc_);
} }
Address RelocInfo::target_address_address() { Address RelocInfo::target_address_address() {
ASSERT(IsCodeTarget(rmode_) || rmode_ == RUNTIME_ENTRY ASSERT(IsCodeTarget(rmode_) || IsRuntimeEntry(rmode_)
|| rmode_ == EMBEDDED_OBJECT || rmode_ == EMBEDDED_OBJECT
|| rmode_ == EXTERNAL_REFERENCE); || rmode_ == EXTERNAL_REFERENCE);
return reinterpret_cast<Address>(Assembler::target_pointer_address_at(pc_)); return reinterpret_cast<Address>(Assembler::target_pointer_address_at(pc_));
...@@ -126,7 +126,7 @@ int RelocInfo::target_address_size() { ...@@ -126,7 +126,7 @@ int RelocInfo::target_address_size() {
void RelocInfo::set_target_address(Address target, WriteBarrierMode mode) { void RelocInfo::set_target_address(Address target, WriteBarrierMode mode) {
ASSERT(IsCodeTarget(rmode_) || rmode_ == RUNTIME_ENTRY); ASSERT(IsCodeTarget(rmode_) || IsRuntimeEntry(rmode_));
Assembler::set_target_address_at(pc_, target); Assembler::set_target_address_at(pc_, target);
if (mode == UPDATE_WRITE_BARRIER && host() != NULL && IsCodeTarget(rmode_)) { if (mode == UPDATE_WRITE_BARRIER && host() != NULL && IsCodeTarget(rmode_)) {
Object* target_code = Code::GetCodeFromTargetAddress(target); Object* target_code = Code::GetCodeFromTargetAddress(target);
...@@ -321,7 +321,7 @@ void RelocInfo::Visit(ObjectVisitor* visitor) { ...@@ -321,7 +321,7 @@ void RelocInfo::Visit(ObjectVisitor* visitor) {
Isolate::Current()->debug()->has_break_points()) { Isolate::Current()->debug()->has_break_points()) {
visitor->VisitDebugTarget(this); visitor->VisitDebugTarget(this);
#endif #endif
} else if (mode == RelocInfo::RUNTIME_ENTRY) { } else if (RelocInfo::IsRuntimeEntry(mode)) {
visitor->VisitRuntimeEntry(this); visitor->VisitRuntimeEntry(this);
} }
} }
...@@ -348,7 +348,7 @@ void RelocInfo::Visit(Heap* heap) { ...@@ -348,7 +348,7 @@ void RelocInfo::Visit(Heap* heap) {
IsPatchedDebugBreakSlotSequence()))) { IsPatchedDebugBreakSlotSequence()))) {
StaticVisitor::VisitDebugTarget(heap, this); StaticVisitor::VisitDebugTarget(heap, this);
#endif #endif
} else if (mode == RelocInfo::RUNTIME_ENTRY) { } else if (RelocInfo::IsRuntimeEntry(mode)) {
StaticVisitor::VisitRuntimeEntry(this); StaticVisitor::VisitRuntimeEntry(this);
} }
} }
......
...@@ -811,7 +811,7 @@ void RelocInfo::Print(FILE* out) { ...@@ -811,7 +811,7 @@ void RelocInfo::Print(FILE* out) {
} }
} else if (IsPosition(rmode_)) { } else if (IsPosition(rmode_)) {
PrintF(out, " (%" V8_PTR_PREFIX "d)", data()); PrintF(out, " (%" V8_PTR_PREFIX "d)", data());
} else if (rmode_ == RelocInfo::RUNTIME_ENTRY && } else if (IsRuntimeEntry(rmode_) &&
Isolate::Current()->deoptimizer_data() != NULL) { Isolate::Current()->deoptimizer_data() != NULL) {
// Depotimization bailouts are stored as runtime entries. // Depotimization bailouts are stored as runtime entries.
int id = Deoptimizer::GetDeoptimizationId( int id = Deoptimizer::GetDeoptimizationId(
......
...@@ -373,7 +373,7 @@ class RelocInfo BASE_EMBEDDED { ...@@ -373,7 +373,7 @@ class RelocInfo BASE_EMBEDDED {
// Read/modify the code target in the branch/call instruction // Read/modify the code target in the branch/call instruction
// this relocation applies to; // this relocation applies to;
// can only be called if IsCodeTarget(rmode_) || rmode_ == RUNTIME_ENTRY // can only be called if IsCodeTarget(rmode_) || IsRuntimeEntry(rmode_)
INLINE(Address target_address()); INLINE(Address target_address());
INLINE(void set_target_address(Address target, INLINE(void set_target_address(Address target,
WriteBarrierMode mode = UPDATE_WRITE_BARRIER)); WriteBarrierMode mode = UPDATE_WRITE_BARRIER));
......
...@@ -282,7 +282,7 @@ static int DecodeIt(Isolate* isolate, ...@@ -282,7 +282,7 @@ static int DecodeIt(Isolate* isolate,
if (rmode == RelocInfo::CODE_TARGET_WITH_ID) { if (rmode == RelocInfo::CODE_TARGET_WITH_ID) {
out.AddFormatted(" (id = %d)", static_cast<int>(relocinfo.data())); out.AddFormatted(" (id = %d)", static_cast<int>(relocinfo.data()));
} }
} else if (rmode == RelocInfo::RUNTIME_ENTRY && } else if (RelocInfo::IsRuntimeEntry(rmode) &&
isolate->deoptimizer_data() != NULL) { isolate->deoptimizer_data() != NULL) {
// A runtime entry reloinfo might be a deoptimization bailout. // A runtime entry reloinfo might be a deoptimization bailout.
Address addr = relocinfo.target_address(); Address addr = relocinfo.target_address();
......
...@@ -139,14 +139,14 @@ void RelocInfo::apply(intptr_t delta) { ...@@ -139,14 +139,14 @@ void RelocInfo::apply(intptr_t delta) {
Address RelocInfo::target_address() { Address RelocInfo::target_address() {
ASSERT(IsCodeTarget(rmode_) || rmode_ == RUNTIME_ENTRY); ASSERT(IsCodeTarget(rmode_) || IsRuntimeEntry(rmode_));
return Assembler::target_address_at(pc_); return Assembler::target_address_at(pc_);
} }
Address RelocInfo::target_address_address() { Address RelocInfo::target_address_address() {
ASSERT(IsCodeTarget(rmode_) || ASSERT(IsCodeTarget(rmode_) ||
rmode_ == RUNTIME_ENTRY || IsRuntimeEntry(rmode_) ||
rmode_ == EMBEDDED_OBJECT || rmode_ == EMBEDDED_OBJECT ||
rmode_ == EXTERNAL_REFERENCE); rmode_ == EXTERNAL_REFERENCE);
// Read the address of the word containing the target_address in an // Read the address of the word containing the target_address in an
...@@ -174,7 +174,7 @@ int RelocInfo::target_address_size() { ...@@ -174,7 +174,7 @@ int RelocInfo::target_address_size() {
void RelocInfo::set_target_address(Address target, WriteBarrierMode mode) { void RelocInfo::set_target_address(Address target, WriteBarrierMode mode) {
ASSERT(IsCodeTarget(rmode_) || rmode_ == RUNTIME_ENTRY); ASSERT(IsCodeTarget(rmode_) || IsRuntimeEntry(rmode_));
Assembler::set_target_address_at(pc_, target); Assembler::set_target_address_at(pc_, target);
if (mode == UPDATE_WRITE_BARRIER && host() != NULL && IsCodeTarget(rmode_)) { if (mode == UPDATE_WRITE_BARRIER && host() != NULL && IsCodeTarget(rmode_)) {
Object* target_code = Code::GetCodeFromTargetAddress(target); Object* target_code = Code::GetCodeFromTargetAddress(target);
...@@ -372,7 +372,7 @@ void RelocInfo::Visit(ObjectVisitor* visitor) { ...@@ -372,7 +372,7 @@ void RelocInfo::Visit(ObjectVisitor* visitor) {
Isolate::Current()->debug()->has_break_points()) { Isolate::Current()->debug()->has_break_points()) {
visitor->VisitDebugTarget(this); visitor->VisitDebugTarget(this);
#endif #endif
} else if (mode == RelocInfo::RUNTIME_ENTRY) { } else if (RelocInfo::IsRuntimeEntry(mode)) {
visitor->VisitRuntimeEntry(this); visitor->VisitRuntimeEntry(this);
} }
} }
...@@ -399,7 +399,7 @@ void RelocInfo::Visit(Heap* heap) { ...@@ -399,7 +399,7 @@ void RelocInfo::Visit(Heap* heap) {
IsPatchedDebugBreakSlotSequence()))) { IsPatchedDebugBreakSlotSequence()))) {
StaticVisitor::VisitDebugTarget(heap, this); StaticVisitor::VisitDebugTarget(heap, this);
#endif #endif
} else if (mode == RelocInfo::RUNTIME_ENTRY) { } else if (RelocInfo::IsRuntimeEntry(mode)) {
StaticVisitor::VisitRuntimeEntry(this); StaticVisitor::VisitRuntimeEntry(this);
} }
} }
......
...@@ -9110,7 +9110,7 @@ void Code::PrintDeoptLocation(int bailout_id) { ...@@ -9110,7 +9110,7 @@ void Code::PrintDeoptLocation(int bailout_id) {
} else if (last_comment != NULL && } else if (last_comment != NULL &&
bailout_id == Deoptimizer::GetDeoptimizationId( bailout_id == Deoptimizer::GetDeoptimizationId(
info->target_address(), Deoptimizer::EAGER)) { info->target_address(), Deoptimizer::EAGER)) {
CHECK(info->rmode() == RelocInfo::RUNTIME_ENTRY); CHECK(RelocInfo::IsRuntimeEntry(info->rmode()));
PrintF(" %s\n", last_comment); PrintF(" %s\n", last_comment);
return; return;
} }
......
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