kpn marked an inline comment as done. kpn added inline comments.
================ Comment at: clang/include/clang/Driver/Options.td:1246 HelpText<"Enables an experimental new pass manager in LLVM.">; +def fexperimental_strict_floating_point : Flag<["-"], "fexperimental-strict-floating-point">, + Group<f_clang_Group>, Flags<[CC1Option]>, ---------------- dang wrote: > A bit late to the party, but can you mark this as > `MarshallingInfoFlag<"LangOpts->ExpStrictFP", "false>` so that `if > (Args.hasArg(OPT_fexperimental_strict_floating_point)) Opts.ExpStrictFP = > true;` in CompilerInvocation.cpp gets generated automatically, we also get > serializing the option for free this way. Assuming this patch makes it to the afternoon, how about I open a new review for this? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80952/new/ https://reviews.llvm.org/D80952 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits