[PATCH] D143675: Discussion: Darwin Sanitizers Stable ABI

Roy Sundahl via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 20 20:11:02 PDT 2023


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



More information about the cfe-commits mailing list