aboutsummaryrefslogtreecommitdiff
path: root/compiler_wrapper/sanitizer_flags.go
diff options
context:
space:
mode:
authorStephen Hines <srhines@google.com>2022-01-06 08:18:42 +0000
committerAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>2022-01-06 08:18:42 +0000
commit38574f5e9831e7f0f28f973b5e61553f5fdb4d78 (patch)
treecf4cdad04e1c146844e2a790910cf9cd1b97082a /compiler_wrapper/sanitizer_flags.go
parentf103b9f78baa21b4edf25d5dd5a1226b6c0ff159 (diff)
parent6323407a0d9e34a73fa3cfb9925f9a16333b4e68 (diff)
downloadtoolchain-utils-38574f5e9831e7f0f28f973b5e61553f5fdb4d78.tar.gz
Merging 32 commit(s) from Chromium's toolchain-utils am: 7bc245bad9 am: 6323407a0d
Original change: https://android-review.googlesource.com/c/platform/external/toolchain-utils/+/1937961 Change-Id: Ib3d45bbdce54e181b3f361afb9af129256ecb101
Diffstat (limited to 'compiler_wrapper/sanitizer_flags.go')
-rw-r--r--compiler_wrapper/sanitizer_flags.go19
1 files changed, 1 insertions, 18 deletions
diff --git a/compiler_wrapper/sanitizer_flags.go b/compiler_wrapper/sanitizer_flags.go
index fe8d1503..da0a64b3 100644
--- a/compiler_wrapper/sanitizer_flags.go
+++ b/compiler_wrapper/sanitizer_flags.go
@@ -9,20 +9,13 @@ import (
)
func processSanitizerFlags(builder *commandBuilder) {
- hasCoverageFlags := false
hasSanitizeFlags := false
- hasSanitizeFuzzerFlags := false
for _, arg := range builder.args {
// TODO: This should probably be -fsanitize= to not match on
- // e.g. -fsanitize-blacklist
+ // e.g. -fsanitize-blocklist
if arg.fromUser {
if strings.HasPrefix(arg.value, "-fsanitize") {
hasSanitizeFlags = true
- if strings.Contains(arg.value, "fuzzer") {
- hasSanitizeFuzzerFlags = true
- }
- } else if arg.value == "-fprofile-instr-generate" {
- hasCoverageFlags = true
}
}
}
@@ -44,15 +37,5 @@ func processSanitizerFlags(builder *commandBuilder) {
}
return arg.value
})
- if builder.target.compilerType == clangType {
- // hasSanitizeFlags && hasCoverageFlags is to work around crbug.com/1013622
- if hasSanitizeFuzzerFlags || (hasSanitizeFlags && hasCoverageFlags) {
- fuzzerFlagsToAdd := []string{
- // TODO: This flag should be removed once fuzzer works with new pass manager
- "-fno-experimental-new-pass-manager",
- }
- builder.addPreUserArgs(fuzzerFlagsToAdd...)
- }
- }
}
}