[clang] 1b6d29e - [Driver] Fix BooleanFFlag identifiers to use 'f' 'fno_' prefixes instead of suffixes

Fangrui Song via cfe-commits cfe-commits at lists.llvm.org
Sat May 30 15:42:01 PDT 2020


Author: Fangrui Song
Date: 2020-05-30T15:41:38-07:00
New Revision: 1b6d29e06b07e518025b6f06445ad3275d6f5684

URL: https://github.com/llvm/llvm-project/commit/1b6d29e06b07e518025b6f06445ad3275d6f5684
DIFF: https://github.com/llvm/llvm-project/commit/1b6d29e06b07e518025b6f06445ad3275d6f5684.diff

LOG: [Driver] Fix BooleanFFlag identifiers to use 'f' 'fno_' prefixes instead of suffixes

Added: 
    

Modified: 
    clang/include/clang/Driver/Options.td

Removed: 
    


################################################################################
diff  --git a/clang/include/clang/Driver/Options.td b/clang/include/clang/Driver/Options.td
index e88e6cf8a130..729cbfb6ad4a 100644
--- a/clang/include/clang/Driver/Options.td
+++ b/clang/include/clang/Driver/Options.td
@@ -3316,11 +3316,9 @@ def Z_reserved_lib_cckext : Flag<["-"], "Z-reserved-lib-cckext">,
     Flags<[LinkerInput, NoArgumentUnused, Unsupported]>, Group<reserved_lib_Group>;
 
 // Ignored options
-// FIXME: multiclasess produce suffixes, not prefixes. This is fine for now
-// since it is only used in ignored options.
 multiclass BooleanFFlag<string name> {
-  def _f : Flag<["-"], "f"#name>;
-  def _fno : Flag<["-"], "fno-"#name>;
+  def f#NAME : Flag<["-"], "f"#name>;
+  def fno_#NAME : Flag<["-"], "fno-"#name>;
 }
 
 defm : BooleanFFlag<"keep-inline-functions">, Group<clang_ignored_gcc_optimization_f_Group>;


        


More information about the cfe-commits mailing list