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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to