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

Unified Diff: src/IceOperand.cpp

Issue 1676123002: Subzero: Use a proper RegNumT type instead of int32_t/SizeT. (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Fix int32_t ==> int for the result of BitVector find_first() and find_next() Created 4 years, 10 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/IceOperand.cpp
diff --git a/src/IceOperand.cpp b/src/IceOperand.cpp
index ff1de640e9a69c47d70085bd216a3a64443c9e55..2e85276b642c76b6d4e12c29014777ed612342ca 100644
--- a/src/IceOperand.cpp
+++ b/src/IceOperand.cpp
@@ -190,7 +190,7 @@ IceString Variable::getName(const Cfg *Func) const {
return "__" + std::to_string(getIndex());
}
-const Variable *Variable::asType(Type Ty, int32_t NewRegNum) const {
+const Variable *Variable::asType(Type Ty, RegNumT NewRegNum) const {
// Note: This returns a Variable, even if the "this" object is a subclass of
// Variable.
if (!BuildDefs::dump() || getType() == Ty)
@@ -198,7 +198,7 @@ const Variable *Variable::asType(Type Ty, int32_t NewRegNum) const {
Variable *V = new (getCurrentCfgAllocator()->Allocate<Variable>())
Variable(kVariable, Ty, Number);
V->NameIndex = NameIndex;
- V->RegNum = NewRegNum == NoRegister ? RegNum : NewRegNum;
+ V->RegNum = NewRegNum == RegNumT::NoRegister ? RegNum : NewRegNum;
V->StackOffset = StackOffset;
return V;
}
@@ -511,8 +511,8 @@ void Variable::dump(const Cfg *Func, Ostream &Str) const {
} else if (Func->getTarget()->hasComputedFrame()) {
if (Func->isVerbose(IceV_RegOrigins))
Str << ":";
- int32_t BaseRegisterNumber = getBaseRegNum();
- if (BaseRegisterNumber == NoRegister)
+ RegNumT BaseRegisterNumber = getBaseRegNum();
+ if (BaseRegisterNumber == RegNumT::NoRegister)
BaseRegisterNumber = Func->getTarget()->getFrameOrStackReg();
Str << "["
<< Func->getTarget()->getRegName(BaseRegisterNumber, IceType_i32);

Powered by Google App Engine
This is Rietveld 408576698