Function Layout, Global Variable Layout and Pooled Constants Layout Reordering

src/IceClFlags.h

 //===- subzero/src/IceClFlags.h - Cl Flags for translation ------*- C++ -*-===//
 //
 //                        The Subzero Code Generator
 //
 // This file is distributed under the University of Illinois Open Source
 // License. See LICENSE.TXT for details.
 //
 //===----------------------------------------------------------------------===//
 //
 // This file declares command line flags controlling translation.
@@ skipping 121 matching lines @@
 
   VerboseMask getVerbose() const {
     return ALLOW_DUMP ? VMask : (VerboseMask)IceV_None;
   }
   void setVerbose(VerboseMask NewValue) { VMask = NewValue; }
 
   void
   setRandomizeAndPoolImmediatesOption(RandomizeAndPoolImmediatesEnum Option) {
     RandomizeAndPoolImmediatesOption = Option;
   }
-
   RandomizeAndPoolImmediatesEnum getRandomizeAndPoolImmediatesOption() const {
     return RandomizeAndPoolImmediatesOption;
   }
 
   void setRandomizeAndPoolImmediatesThreshold(uint32_t Threshold) {
     RandomizeAndPoolImmediatesThreshold = Threshold;
   }
   uint32_t getRandomizeAndPoolImmediatesThreshold() const {
     return RandomizeAndPoolImmediatesThreshold;
   }
 
+  void setShouldReorderFunctions(bool Option) { ReorderFunctions = Option; }
+  bool shouldReorderFunctions() const { return ReorderFunctions; }
+
+  void setReorderFunctionsWindowSize(uint32_t Size) {
+    ReorderFunctionsWindowSize = Size;
+  }
+  uint32_t getReorderFunctionsWindowSize() const {
+    return ReorderFunctionsWindowSize;
+  }
+
+  void setShouldReorderGlobalVariables(bool Option) {
+    ReorderGlobalVariables = Option;
+  }
+  bool shouldReorderGlobalVariables() const { return ReorderGlobalVariables; }
+
+  void setShouldReorderPooledConstants(bool Option) {
+    ReorderPooledConstants = Option;
+  }
+  bool shouldReorderPooledConstants() const { return ReorderPooledConstants; }
+
   // IceString accessors.
 
   const IceString &getDefaultFunctionPrefix() const {
     return DefaultFunctionPrefix;
   }
   void setDefaultFunctionPrefix(const IceString &NewValue) {
     DefaultFunctionPrefix = NewValue;
   }
 
   const IceString &getDefaultGlobalPrefix() const {
@@ skipping 36 matching lines @@
   bool DisableInternal;
   bool DisableIRGeneration;
   bool DisableTranslation;
   bool DumpStats;
   bool EnableBlockProfile;
   bool FunctionSections;
   bool GenerateUnitTestMessages;
   bool PhiEdgeSplit;
   bool RandomNopInsertion;
   bool RandomRegAlloc;
+  bool ReorderFunctions;
+  bool ReorderGlobalVariables;
+  bool ReorderPooledConstants;
   bool SkipUnimplemented;
   bool SubzeroTimingEnabled;
   bool TimeEachFunction;
   bool UseSandboxing;
 
   OptLevel Opt;
   FileType OutFileType;
+  size_t NumTranslationThreads; // 0 means completely sequential

[jvoung (off chromium), 2015/06/24 21:26:23]

Let's leave size_t and the uint64_t where it was before. size_t is sometimes
64-bit.

[qining, 2015/06/24 23:22:33]

Done.

+  RandomizeAndPoolImmediatesEnum RandomizeAndPoolImmediatesOption;
+  uint32_t RandomizeAndPoolImmediatesThreshold;
   int RandomMaxNopsPerInstruction;
   int RandomNopProbabilityAsPercentage;
+  uint64_t RandomSeed;

[jvoung (off chromium), 2015/06/24 21:26:23]

Same -- Otherwise this is interleaving 32-bit things with 64-bit things, which
will leave padding gaps in between, unless you are lucky and end up with an even
number of 32-bit things before the 64-bit thing.

[qining, 2015/06/24 23:22:33]

Done.

+  uint32_t ReorderFunctionsWindowSize;
   TargetArch TArch;
   TargetInstructionSet TInstrSet;
   VerboseMask VMask;
 
   IceString DefaultFunctionPrefix;
   IceString DefaultGlobalPrefix;
   IceString TestPrefix;
   IceString TimingFocusOn;
   IceString TranslateOnly;
   IceString VerboseFocusOn;
-
-  // Immediates Randomization and Pooling options
-  RandomizeAndPoolImmediatesEnum RandomizeAndPoolImmediatesOption;
-  uint32_t RandomizeAndPoolImmediatesThreshold;
-
-  size_t NumTranslationThreads; // 0 means completely sequential
-  uint64_t RandomSeed;
 };
 
 } // end of namespace Ice
 
 #endif // SUBZERO_SRC_ICECLFLAGS_H