eugenis added inline comments.

================
Comment at: lib/Frontend/CompilerInvocation.cpp:735
+                   OPT_fno_sanitize_thread_data_races,
+                   Opts.SanitizeThreadDataRaces);
+  Opts.SanitizeThreadStackTraces =
----------------
alekseyshl wrote:
> eugenis wrote:
> > It seems common to hardcode the default option value here.
> > The same in SanitizerArgs.cpp.
> True, but the default value for these flags is already mentioned in more than 
> one place, why not to try to reduce the complexity?
I guess I don't mind either way.


https://reviews.llvm.org/D26461



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to