[llvm] [ASan] Allow for passing AddressSanitizer command line options through the AddressSanitizerOptions struct. (PR #72439)

Usama Hameed via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 27 11:59:11 PST 2023


================
@@ -20,12 +20,20 @@ namespace llvm {
 class Module;
 class raw_ostream;
 
+static const int InstrumentationWithCallsThreshold = 7000;
----------------
usama54321 wrote:

I want to use the same default values in both AddressSanitizerOptions and the cl::init() calls. If I put these in the struct directly, I can't reference them in the cl::init calls which creates the chance of a mismatch between these two places which could lead to weird bugs.

Do you think adding a new namespace here makes sense? Or maybe the mismatch is not a big concern.

https://github.com/llvm/llvm-project/pull/72439


More information about the llvm-commits mailing list