[clang] [clang] Add deprecation warning for `-Ofast` driver option (PR #98736)
Shafik Yaghmour via cfe-commits
cfe-commits at lists.llvm.org
Wed Jul 17 20:01:40 PDT 2024
================
@@ -931,7 +931,9 @@ def O : Joined<["-"], "O">, Group<O_Group>,
def O_flag : Flag<["-"], "O">, Visibility<[ClangOption, CC1Option, FC1Option]>,
Alias<O>, AliasArgs<["1"]>;
def Ofast : Joined<["-"], "Ofast">, Group<O_Group>,
- Visibility<[ClangOption, CC1Option, FlangOption]>;
+ Visibility<[ClangOption, CC1Option, FlangOption]>,
+ HelpText<"Deprecated; use '-O3 -ffast math' for the same behavior,"
+ " or '-O3' to enable conforming optimizations only">;
----------------
shafik wrote:
```suggestion
" or '-O3' to enable only conforming optimizations">;
```
I apologize for the late nitpick here but this feels better worded to me since the only applies to enable.
https://github.com/llvm/llvm-project/pull/98736
More information about the cfe-commits
mailing list