jansvoboda11 requested changes to this revision. jansvoboda11 added a comment. This revision now requires changes to proceed.
Hi Kevin, sorry for the late response. I don't think adding new tests is necessary as long as the flag is used in existing ones. ================ Comment at: clang/include/clang/Driver/Options.td:1286 + HelpText<"Enables experimental strict floating point in LLVM.">, + MarshallingInfoFlag<"LangOpts->ExpStrictFP", "false">; def finput_charset_EQ : Joined<["-"], "finput-charset=">, Group<f_Group>; ---------------- In D82756, we've replaced the "default value" argument with a list of options that can imply the current flag. If empty or omitted, we set the default value to `false` automatically. Could you please remove the `"false"` argument? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88987/new/ https://reviews.llvm.org/D88987 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
