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

Unified Diff: src/IceOperand.cpp

Issue 2177033002: Subzero: Local variable splitting. (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Cleanup Created 4 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/IceOperand.cpp
diff --git a/src/IceOperand.cpp b/src/IceOperand.cpp
index 540252b64785faa14f8ffecdbc8ef4689f75cea0..d3c019287e86624a8f1a57ec87b28683da125e9f 100644
--- a/src/IceOperand.cpp
+++ b/src/IceOperand.cpp
@@ -208,10 +208,11 @@ const Variable *Variable::asType(const Cfg *Func, Type Ty,
}
RegWeight Variable::getWeight(const Cfg *Func) const {
- VariablesMetadata *VMetadata = Func->getVMetadata();
- return mustHaveReg() ? RegWeight(RegWeight::Inf)
- : mustNotHaveReg() ? RegWeight(RegWeight::Zero)
- : VMetadata->getUseWeight(this);
+ if (mustHaveReg())
+ return RegWeight(RegWeight::Inf);
+ if (mustNotHaveReg())
+ return RegWeight(RegWeight::Zero);
+ return Func->getVMetadata()->getUseWeight(this);
}
void VariableTracking::markUse(MetadataKind TrackingKind, const Inst *Instr,
@@ -540,8 +541,10 @@ void Variable::dump(const Cfg *Func, Ostream &Str) const {
if (Func->isVerbose(IceV_RegOrigins) ||
(!hasReg() && !Func->getTarget()->hasComputedFrame())) {
Str << "%" << getName();
- if (getLinkedTo() != nullptr)
- Str << ":%" << getLinkedTo()->getName();
+ for (Variable *Link = getLinkedTo(); Link != nullptr;
+ Link = Link->getLinkedTo()) {
+ Str << ":%" << Link->getName();
+ }
}
if (hasReg()) {
if (Func->isVerbose(IceV_RegOrigins))
« src/IceOperand.h ('K') | « src/IceOperand.h ('k') | src/IceTargetLowering.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698