Chromium Code Reviews| 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; |
| }; |