[PATCH] Common flags in ASan.

[Evgeniy Stepanov]

================
Comment at: lib/asan/asan_rtl.cc:432
@@ -433,3 +431,3 @@
   // Check if external symbolizer is defined before parsing the flags.
-  asan_external_symbolizer = GetEnv("ASAN_SYMBOLIZER_PATH");
+  common_flags()->external_symbolizer_path = GetEnv("ASAN_SYMBOLIZER_PATH");
   // Initialize flags. This must be done early, because most of the
----------------
Please move this to InitializeFlags.

================
Comment at: lib/asan/asan_stack.cc:17
@@ -16,2 +16,3 @@
 #include "asan_stack.h"
+#include "sanitizer_common/sanitizer_flags.h"
 
----------------
You seem to be including this way too often. Can it be done in asan_flags.h? Why asan_flags.h is not included this often?

================
Comment at: lib/sanitizer_common/sanitizer_flags.h:26
@@ +25,3 @@
+struct CommonFlags {
+  // If set, use in-process symbolizer from common sanitizer runtime.
+  bool symbolize;
----------------
This comment looks out of date?


http://llvm-reviews.chandlerc.com/D747