Subzero: Fix symbol name mangling. Make flags global.

src/IceClFlags.h

Index: src/IceClFlags.h
diff --git a/src/IceClFlags.h b/src/IceClFlags.h
index d3e8b26510b512454b8bdd50afa03c7bad7dc18f..d10c8c4d26bfb5c5f0b342e11726053ad196ae8a 100644
--- a/src/IceClFlags.h
+++ b/src/IceClFlags.h
@@ -20,6 +20,21 @@
 
 namespace Ice {
 
+// TODO(stichnot): Fix the separation between ClFlags and ClFlagsExtra.
+//
+// The original intention was that ClFlags would be the core set of flags for a
+// release build, while ClFlagsExtra had optional flags that would be locked to
+// default values in a release build.  However, the division has evolved to be
+// fairly arbitrary.
+//
+// The variable flags in a release (browser) build should be limited to opt
+// level, number of threads, output file, and perhaps input file.
+//
+// The core flags should remain part of the GlobalContext object, but the
+// optional flags might as well be global, i.e. static members of GlobalContext,
+// so that they are easily accessed from anywhere without needing to plumb in
+// the GlobalContext object.
+
 class ClFlagsExtra;
 
 /// Define variables which configure translation and related support functions.