Subzero: Initial implementation of BB Local CSE

src/IceClFlags.def

 //===- subzero/src/IceClFlags.def - 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.
 //
 //===----------------------------------------------------------------------===//
 ///
 /// \file
@@ skipping 122 matching lines @@
   X(DumpStrings, bool, dev_opt_flag,                                           \
     "dump-strings",                                                            \
     cl::desc("Dump string pools during compilation"),                          \
     cl::init(false))                                                           \
                                                                                \
   X(EnableBlockProfile, bool, dev_opt_flag, "enable-block-profile",            \
     cl::desc("Instrument basic blocks, and output profiling "                  \
              "information to stdout at the end of program execution."),        \
     cl::init(false))                                                           \
                                                                                \
+  X(SubzeroExperimentalOptsEnabled, bool, dev_opt_flag, "experimental-opts",   \

[Jim Stichnoth, 2016/05/19 01:35:50]

I think you can remove the "Subzero" prefix, because that is pretty much assumed
here.

And to keep a consistent naming style, I would suggest something like
"-enable-experimental" and name the variable something like EnableExperimental.

[manasijm, 2016/05/19 17:50:16]

Done.

+    cl::desc("Enable Optimizations not yet part of O2"),                       \
+    cl::init(false))                                                           \
+                                                                               \
   X(EnablePhiEdgeSplit, bool, dev_opt_flag, "phi-edge-split",                  \
     cl::desc("Enable edge splitting for Phi lowering"), cl::init(true))        \
                                                                                \
   X(ExcludedRegisters, std::string, dev_list_flag, "reg-exclude",              \
     cl::CommaSeparated, cl::desc("Don't use specified registers"))             \
                                                                                \
   X(ForceMemIntrinOpt, bool, dev_opt_flag, "fmem-intrin-opt",                  \
     cl::desc("Force optimization of memory intrinsics."))                      \
                                                                                \
   X(ForceO2String, std::string, dev_opt_flag, "force-O2",                      \
@@ skipping 195 matching lines @@
     cl::init(":"))                                                             \
                                                                                \
   X(WasmBoundsCheck, bool, dev_opt_flag, "wasm-bounds-check",                  \
     cl::desc("Add bounds checking code in WASM frontend"),                     \
     cl::init(true))
 //#define X(Name, Type, ClType, ...)
 
 } // end of namespace Ice
 
 #endif // SUBZERO_SRC_ICECLFLAGS_DEF