We currently test the leak detector by running "go build -asan",
which will pass -fsanitize=address to the C compiler.
So use that when testing whether the option works.
Fixes #72128
Change-Id: I4efc0b689bfda04c80dbac30a5c757215f297d2f
Reviewed-on: https://go-review.googlesource.com/c/go/+/655535
Reviewed-by: Ian Lance Taylor <iant@google.com>
Reviewed-by: Cherry Mui <cherryyz@google.com>
Auto-Submit: Ian Lance Taylor <iant@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
return c
}
+ sanitizerOpt := sanitizer
+ // For the leak detector, we use "go build -asan",
+ // which implies the address sanitizer.
+ // We may want to adjust this someday.
+ if sanitizer == "leak" {
+ sanitizerOpt = "address"
+ }
+
c := &config{
sanitizer: sanitizer,
- cFlags: []string{"-fsanitize=" + sanitizer},
- ldFlags: []string{"-fsanitize=" + sanitizer},
+ cFlags: []string{"-fsanitize=" + sanitizerOpt},
+ ldFlags: []string{"-fsanitize=" + sanitizerOpt},
}
if testing.Verbose() {