[PATCH] D45289: Disable -fmerge-all-constants as default.

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 4 15:40:36 PDT 2018


rjmccall added inline comments.


================
Comment at: include/clang/Driver/Options.td:1286
 def fno_merge_all_constants : Flag<["-"], "fno-merge-all-constants">, Group<f_Group>,
     Flags<[CC1Option]>, HelpText<"Disallow merging of constants">;
 def fno_modules : Flag <["-"], "fno-modules">, Group<f_Group>,
----------------
rsmith wrote:
> Remove the `Flags<[CC1Option]>` here, since `-cc1` no longer understands this flag.
Did you mean to remove the help text here?  It's still a meaningful driver option.


Repository:
  rC Clang

https://reviews.llvm.org/D45289





More information about the cfe-commits mailing list