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

Unified Diff: src/IceUtils.h

Issue 1683243003: ARM32 Vector lowering - scalarize select (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Better determinism and fixed nits. 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/IceUtils.h
diff --git a/src/IceUtils.h b/src/IceUtils.h
index f9b03bc75b1f65da97b6f1dc5494750b36af500f..9efd44c3a28b2a991844d7c72ad2168e05afd7ef 100644
--- a/src/IceUtils.h
+++ b/src/IceUtils.h
@@ -123,6 +123,25 @@ template <typename T> static bool isPositiveZero(T Val) {
return Val == 0 && !std::signbit(Val);
}
+/// A helper class to ensure that a boolean flag is restored to its previous
Jim Stichnoth 2016/02/13 05:22:09 Could you add "RAII" to the comment somewhere? Th
Eric Holk 2016/02/16 23:05:30 Sure.
+/// value upon function exist.
+///
+/// Used in places like RandomizationPoolingPause and generating target helper
+/// calls.
+class BoolFlagSaver {
+ BoolFlagSaver() = delete;
+ BoolFlagSaver(const BoolFlagSaver &) = delete;
+ BoolFlagSaver &operator=(const BoolFlagSaver &) = delete;
+
+public:
+ BoolFlagSaver(bool &F, bool NewValue) : OldValue(F), Flag(F) { F = NewValue; }
+ ~BoolFlagSaver() { Flag = OldValue; }
+
+private:
+ const bool OldValue;
+ bool &Flag;
+};
+
} // end of namespace Utils
} // end of namespace Ice

Powered by Google App Engine
This is Rietveld 408576698