rsundahl marked an inline comment as done.
rsundahl added inline comments.

================
Comment at: clang/include/clang/Driver/Options.td:1785
                                                    HelpText<"Use default code 
inlining logic for the address sanitizer">;
+def fsanitize_address_stable_abi : Flag<["-"], "fsanitize-address-stable-abi">,
+                                                Group<f_clang_Group>,
----------------
vitalybuka wrote:
> how likely you will need thus for  other sanitizers in future
> should this be rather -fsanitize-stable-abi which is ignore for now for other 
> sanitizers?
Thanks @vitalybuka. I agree and made this change. For now I still only consume 
the flag if sanitizer=address so that we continue to get the unused option 
warning in the other cases and left the test for that warning intact.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143675

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

Reply via email to