[PATCH] D30920: Do not pass -Os and -Oz to the Gold plugin

Pirama Arumuga Nainar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 13 21:25:41 PDT 2017


pirama marked 3 inline comments as done.
pirama added inline comments.


================
Comment at: lib/Driver/ToolChains/CommonArgs.cpp:369
     if (A->getOption().matches(options::OPT_O4) ||
-        A->getOption().matches(options::OPT_Ofast))
+        A->getOption().matches(options::OPT_Ofast)) {
       OOpt = "3";
----------------
tejohnson wrote:
> Remove added "{"
Done.  I thought having braces consistent across `if` and `else` branches would be a consistent coding style but doesn't seem so: clang-format doesn't do this.


https://reviews.llvm.org/D30920





More information about the cfe-commits mailing list