================
@@ -2595,6 +2595,16 @@ def fsanitize_undefined_trap_on_error
 def fno_sanitize_undefined_trap_on_error
     : Flag<["-"], "fno-sanitize-undefined-trap-on-error">, 
Group<f_clang_Group>,
       Alias<fno_sanitize_trap_EQ>, AliasArgs<["undefined"]>;
+defm sanitize_annotate_debug_info_traps
+    : BoolFOption<
+          "sanitize-annotate-debug-info-traps",
----------------
delcypher wrote:

So I suggested the name `-fsanitize-annotate-debug-info-traps`. The reason I 
picked that name was

* The feature annotates the debug info on traps. The proposed flag name 
describes this.
* The feature is very similar conceptually to `-fsanitize-annotate-debug-info` 
because that feature also annotates the debug info. We did consider different 
names but came to the conclusion that is potentially very confusing to have two 
different flags that both annotate the debug info but use completely different 
terminology for that in that flag name.

One alternative could be something like `-fsanitize-debug-trap-reasons`. So the 
name here doesn't really say anything about the implementation (annotating 
debug info) and more about what it tries to do, i.e. help with debugging the 
reason for trapping.

@thurstond Do you prefer `-fsanitize-debug-trap-reasons` or have another 
suggestion?


https://github.com/llvm/llvm-project/pull/145967
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to