[PATCH] D88987: [FPEnv][Clang][Driver] Use MarshallingInfoFlag for -fexperimental-strict-floating-point

[Kevin P. Neal via Phabricator via cfe-commits]

kpn added a comment.

No worries!

The option does already have tests that verify correct operation.



================
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>;
----------------
jansvoboda11 wrote:
> 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?
Will do.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88987/new/

https://reviews.llvm.org/D88987