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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits