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

Unified Diff: src/IceAssemblerX86BaseImpl.h

Issue 1257283004: Iasm and obj lowering for advanced switch lowering. (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Created 5 years, 5 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: src/IceAssemblerX86BaseImpl.h
diff --git a/src/IceAssemblerX86BaseImpl.h b/src/IceAssemblerX86BaseImpl.h
index f8ba4d4fdb498f2448cddd1b242ff4ed72bb9079..0e9d5babe79a8758a25fdab6d224bcbf96486097 100644
--- a/src/IceAssemblerX86BaseImpl.h
+++ b/src/IceAssemblerX86BaseImpl.h
@@ -31,26 +31,31 @@ template <class Machine>
AssemblerX86Base<Machine>::~AssemblerX86Base<Machine>() {
if (BuildDefs::asserts()) {
for (const Label *Label : CfgNodeLabels) {
- Label->FinalCheck();
+ Label->finalCheck();
}
for (const Label *Label : LocalLabels) {
- Label->FinalCheck();
+ Label->finalCheck();
}
}
}
template <class Machine> void AssemblerX86Base<Machine>::alignFunction() {
- SizeT Align = 1 << getBundleAlignLog2Bytes();
+ const SizeT Align = 1 << getBundleAlignLog2Bytes();
SizeT BytesNeeded = Utils::OffsetToAlignment(Buffer.getPosition(), Align);
- const SizeT HltSize = 1;
+ constexpr SizeT HltSize = 1;
while (BytesNeeded > 0) {
hlt();
BytesNeeded -= HltSize;
}
}
+template <class Machine> void AssemblerX86Base<Machine>::alignCfgNode() {
+ const SizeT Align = 1 << getBundleAlignLog2Bytes();
+ padWithNop(Utils::OffsetToAlignment(Buffer.getPosition(), Align));
+}
+
template <class Machine>
-Label *AssemblerX86Base<Machine>::GetOrCreateLabel(SizeT Number,
+Label *AssemblerX86Base<Machine>::getOrCreateLabel(SizeT Number,
LabelVector &Labels) {
Label *L = nullptr;
if (Number == Labels.size()) {
@@ -70,25 +75,25 @@ Label *AssemblerX86Base<Machine>::GetOrCreateLabel(SizeT Number,
}
template <class Machine>
-Label *AssemblerX86Base<Machine>::GetOrCreateCfgNodeLabel(SizeT NodeNumber) {
- return GetOrCreateLabel(NodeNumber, CfgNodeLabels);
+Label *AssemblerX86Base<Machine>::getOrCreateCfgNodeLabel(SizeT NodeNumber) {
+ return getOrCreateLabel(NodeNumber, CfgNodeLabels);
}
template <class Machine>
-Label *AssemblerX86Base<Machine>::GetOrCreateLocalLabel(SizeT Number) {
- return GetOrCreateLabel(Number, LocalLabels);
+Label *AssemblerX86Base<Machine>::getOrCreateLocalLabel(SizeT Number) {
+ return getOrCreateLabel(Number, LocalLabels);
}
template <class Machine>
void AssemblerX86Base<Machine>::bindCfgNodeLabel(SizeT NodeNumber) {
assert(!getPreliminary());
- Label *L = GetOrCreateCfgNodeLabel(NodeNumber);
+ Label *L = getOrCreateCfgNodeLabel(NodeNumber);
this->bind(L);
}
template <class Machine>
void AssemblerX86Base<Machine>::BindLocalLabel(SizeT Number) {
- Label *L = GetOrCreateLocalLabel(Number);
+ Label *L = getOrCreateLocalLabel(Number);
if (!getPreliminary())
this->bind(L);
}
@@ -2780,10 +2785,10 @@ template <class Machine>
void AssemblerX86Base<Machine>::j(typename Traits::Cond::BrCond condition,
Label *label, bool near) {
AssemblerBuffer::EnsureCapacity ensured(&Buffer);
- if (label->IsBound()) {
+ if (label->isBound()) {
static const int kShortSize = 2;
static const int kLongSize = 6;
- intptr_t offset = label->Position() - Buffer.size();
+ intptr_t offset = label->getPosition() - Buffer.size();
assert(offset <= 0);
if (Utils::IsInt(8, offset - kShortSize)) {
// TODO(stichnot): Here and in jmp(), we may need to be more
@@ -2830,10 +2835,10 @@ void AssemblerX86Base<Machine>::jmp(typename Traits::GPRRegister reg) {
template <class Machine>
void AssemblerX86Base<Machine>::jmp(Label *label, bool near) {
AssemblerBuffer::EnsureCapacity ensured(&Buffer);
- if (label->IsBound()) {
+ if (label->isBound()) {
static const int kShortSize = 2;
static const int kLongSize = 5;
- intptr_t offset = label->Position() - Buffer.size();
+ intptr_t offset = label->getPosition() - Buffer.size();
assert(offset <= 0);
if (Utils::IsInt(8, offset - kShortSize)) {
emitUint8(0xEB);
@@ -2959,20 +2964,20 @@ void AssemblerX86Base<Machine>::align(intptr_t alignment, intptr_t offset) {
template <class Machine> void AssemblerX86Base<Machine>::bind(Label *label) {
intptr_t bound = Buffer.size();
- assert(!label->IsBound()); // Labels can only be bound once.
- while (label->IsLinked()) {
- intptr_t position = label->LinkPosition();
+ assert(!label->isBound()); // Labels can only be bound once.
+ while (label->isLinked()) {
+ intptr_t position = label->getLinkPosition();
intptr_t next = Buffer.load<int32_t>(position);
Buffer.store<int32_t>(position, bound - (position + 4));
- label->position_ = next;
+ label->Position = next;
}
- while (label->HasNear()) {
- intptr_t position = label->NearPosition();
+ while (label->hasNear()) {
+ intptr_t position = label->getNearPosition();
intptr_t offset = bound - (position + 1);
assert(Utils::IsInt(8, offset));
Buffer.store<int8_t>(position, offset);
}
- label->BindTo(bound);
+ label->bindTo(bound);
}
template <class Machine>
@@ -3048,8 +3053,8 @@ void AssemblerX86Base<Machine>::emitComplex(
template <class Machine>
void AssemblerX86Base<Machine>::emitLabel(Label *label,
intptr_t instruction_size) {
- if (label->IsBound()) {
- intptr_t offset = label->Position() - Buffer.size();
+ if (label->isBound()) {
+ intptr_t offset = label->getPosition() - Buffer.size();
assert(offset <= 0);
emitInt32(offset - instruction_size);
} else {
@@ -3059,20 +3064,20 @@ void AssemblerX86Base<Machine>::emitLabel(Label *label,
template <class Machine>
void AssemblerX86Base<Machine>::emitLabelLink(Label *Label) {
- assert(!Label->IsBound());
+ assert(!Label->isBound());
intptr_t Position = Buffer.size();
- emitInt32(Label->position_);
+ emitInt32(Label->Position);
if (!getPreliminary())
- Label->LinkTo(Position);
+ Label->linkTo(Position);
}
template <class Machine>
void AssemblerX86Base<Machine>::emitNearLabelLink(Label *label) {
- assert(!label->IsBound());
+ assert(!label->isBound());
intptr_t position = Buffer.size();
emitUint8(0);
if (!getPreliminary())
- label->NearLinkTo(position);
+ label->nearLinkTo(position);
}
template <class Machine>

Powered by Google App Engine
This is Rietveld 408576698