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

Unified Diff: src/IceInst.h

Issue 1141213004: Subzero: Fold icmp into br/select lowering. (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Remove unnecessary break statement Created 5 years, 7 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/IceInst.h
diff --git a/src/IceInst.h b/src/IceInst.h
index e3262c83d8dee23590b63d96981806321ed45bf5..28fb0468576437498910400ec9791dd85b514226 100644
--- a/src/IceInst.h
+++ b/src/IceInst.h
@@ -81,6 +81,7 @@ public:
bool isDeleted() const { return Deleted; }
void setDeleted() { Deleted = true; }
+ void setDead(bool Value = true) { Dead = Value; }
void deleteIfDead();
bool hasSideEffects() const { return HasSideEffects; }
@@ -178,7 +179,9 @@ protected:
InstNumberT Number;
// Deleted means irrevocably deleted.
bool Deleted;
- // Dead means pending deletion after liveness analysis converges.
+ // Dead means one of two things depending on context: (1) pending
+ // deletion after liveness analysis converges, or (2) marked for
+ // deletion during lowering due to a folded bool operation.
bool Dead;
// HasSideEffects means the instruction is something like a function
// call or a volatile load that can't be removed even if its Dest
« no previous file with comments | « src/IceDefs.h ('k') | src/IceTargetLowering.h » ('j') | src/IceTargetLoweringX8632.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698