[probably NFC] Mix all sanitizers into module hash

beccadax · beccadax · commit 47a4505429cd · 2019-08-23T15:20:31.000-07:00
Mixes the state of all sanitizers, not just the fuzzer, into the module hash used to decide whether to skip LLVM codegen. I don’t actually know of a case where one of the other sanitizers will generate identical IR and different machine code, but being defensive costs us very little.
diff --git a/include/swift/AST/IRGenOptions.h b/include/swift/AST/IRGenOptions.h
@@ -265,7 +265,7 @@ class IRGenOptions {
        << 'd' << DisableLLVMOptzns
        << 'D' << DisableSwiftSpecificLLVMOptzns
        << 'p' << GenerateProfile
-       << 's' << Sanitizers.contains(SanitizerKind::Fuzzer);
+       << 's' << Sanitizers.toRaw();
   }
 
   /// Should LLVM IR value names be emitted and preserved?

Original file line number	Diff line number	Diff line change
`@@ -265,7 +265,7 @@ class IRGenOptions {`
`265`	`265`	`<< 'd' << DisableLLVMOptzns`
`266`	`266`	`<< 'D' << DisableSwiftSpecificLLVMOptzns`
`267`	`267`	`<< 'p' << GenerateProfile`
`268`		`- << 's' << Sanitizers.contains(SanitizerKind::Fuzzer);`
	`268`	`+ << 's' << Sanitizers.toRaw();`
`269`	`269`	`}`
`270`	`270`
`271`	`271`	`/// Should LLVM IR value names be emitted and preserved?`