Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(194)

Unified Diff: runtime/vm/code_patcher_x64.cc

Issue 1301963003: VM: Clean up and fix bugs in instructions patterns (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: s/Pattern::InstructionLength/Pattern::pattern_length_in_bytes/g Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: runtime/vm/code_patcher_x64.cc
diff --git a/runtime/vm/code_patcher_x64.cc b/runtime/vm/code_patcher_x64.cc
index 1f3f8e18aae25f6667fa7818dbef9e6f2b4d28f1..58478bfe09c0f5160a4acb04642ea75391cad02e 100644
--- a/runtime/vm/code_patcher_x64.cc
+++ b/runtime/vm/code_patcher_x64.cc
@@ -41,17 +41,17 @@ class UnoptimizedCall : public ValueObject {
}
RawObject* ic_data() const {
- intptr_t index = InstructionPattern::IndexFromPPLoad(start_ + 3);
+ intptr_t index = IndexFromPPLoad(start_ + 3);
return object_pool_.ObjectAt(index);
}
uword target() const {
- intptr_t index = InstructionPattern::IndexFromPPLoad(start_ + 10);
+ intptr_t index = IndexFromPPLoad(start_ + 10);
return object_pool_.RawValueAt(index);
}
void set_target(uword target) const {
- intptr_t index = InstructionPattern::IndexFromPPLoad(start_ + 10);
+ intptr_t index = IndexFromPPLoad(start_ + 10);
object_pool_.SetRawValueAt(index, target);
// No need to flush the instruction cache, since the code is not modified.
}
@@ -117,7 +117,7 @@ class PoolPointerCall : public ValueObject {
}
intptr_t pp_index() const {
- return InstructionPattern::IndexFromPPLoad(start_ + 3);
+ return IndexFromPPLoad(start_ + 3);
}
uword Target() const {
@@ -190,11 +190,11 @@ intptr_t CodePatcher::InstanceCallSizeInBytes() {
void CodePatcher::InsertCallAt(uword start, uword target) {
// The inserted call should not overlap the lazy deopt jump code.
- ASSERT(start + ShortCallPattern::InstructionLength() <= target);
+ ASSERT(start + ShortCallPattern::pattern_length_in_bytes() <= target);
*reinterpret_cast<uint8_t*>(start) = 0xE8;
ShortCallPattern call(start);
call.SetTargetAddress(target);
- CPU::FlushICache(start, ShortCallPattern::InstructionLength());
+ CPU::FlushICache(start, ShortCallPattern::pattern_length_in_bytes());
}
@@ -227,7 +227,7 @@ class EdgeCounter : public ValueObject {
}
RawObject* edge_counter() const {
- return object_pool_.ObjectAt(InstructionPattern::IndexFromPPLoad(end_ - 4));
+ return object_pool_.ObjectAt(IndexFromPPLoad(end_ - 4));
}
private:

Powered by Google App Engine
This is Rietveld 408576698