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

Unified Diff: src/IceClFlags.h

Issue 1185703004: Add constant blinding/pooling option for X8632 code translation (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: reformat Created 5 years, 6 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
« no previous file with comments | « no previous file | src/IceClFlags.cpp » ('j') | src/IceClFlags.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/IceClFlags.h
diff --git a/src/IceClFlags.h b/src/IceClFlags.h
index 7df697303c5d6a5d77cae079e25f88a770743aaf..d87187e77a12a822428e6a31d4d1d48635c1ca5b 100644
--- a/src/IceClFlags.h
+++ b/src/IceClFlags.h
@@ -176,6 +176,22 @@ public:
uint64_t getRandomSeed() const { return RandomSeed; }
void setRandomSeed(size_t NewValue) { RandomSeed = NewValue; }
+ // Immediates Randomization and Pooling options getter and setter
+ void setRandomizeAndPoolImmediatesOption(
+ Ice::RandomizeAndPoolImmediatesEnum option) {
Jim Stichnoth 2015/06/12 23:48:17 capitalize Option per LLVM coding conventions
qining 2015/06/17 04:28:53 Done.
+ RandomizeAndPoolImmediatesOption = option;
+ }
+ bool shouldRandomizeImmediates() const {
Jim Stichnoth 2015/06/12 23:48:16 I think it would be better to have a single getRan
qining 2015/06/17 04:28:53 Done.
+ return RandomizeAndPoolImmediatesOption == Ice::RandomizeImmediates;
+ }
+ bool shouldPoolImmediates() const {
+ return RandomizeAndPoolImmediatesOption == Ice::PoolImmediates;
+ }
+ bool shouldNotRandomizeOrPoolImmediates() const {
+ return RandomizeAndPoolImmediatesOption ==
+ Ice::NoneImmediatesRandomizationPooling;
Jim Stichnoth 2015/06/12 23:48:17 I don't think you need "Ice::" here and above sinc
qining 2015/06/17 04:28:53 Done.
+ }
+
private:
bool AllowErrorRecovery;
bool AllowUninitializedGlobals;
@@ -211,6 +227,9 @@ private:
IceString TranslateOnly;
IceString VerboseFocusOn;
+ // Immediates Randomization and Pooling options
+ RandomizeAndPoolImmediatesEnum RandomizeAndPoolImmediatesOption;
+
size_t NumTranslationThreads; // 0 means completely sequential
uint64_t RandomSeed;
};
« no previous file with comments | « no previous file | src/IceClFlags.cpp » ('j') | src/IceClFlags.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698