1114662 - add --enable-thread-sanitizer configure option

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Nathan Froyd [:froydnj]]

Like --enable-{address,memory}-sanitizer, this doesn't add the
appropriate -fsanitize=FOO flags; it merely sets internal defines and
the LLVM_SYMBOLIZER variable.

Comment 1

[Nathan Froyd [:froydnj]]

Attachment: add --enable-thread-sanitizer configure option

Comment 2

[Nathan Froyd [:froydnj]]

Attachment: add --enable-thread-sanitizer configure option

Like --enable-{address,memory}-sanitizer, this doesn't add the
appropriate -fsanitize=FOO flags; it merely sets internal defines and
the LLVM_SYMBOLIZER variable.

(Now with correct spelling...)

Comment 3

[Nathan Froyd [:froydnj]]

Attachment: add --enable-thread-sanitizer configure option

Like --enable-{address,memory}-sanitizer, this doesn't add the
appropriate -fsanitize=FOO flags; it merely sets internal defines and
the LLVM_SYMBOLIZER variable.

(Sigh, we need to add it to js/src/, too.)

Comment 4

[Mike Hommey [:glandium]]

Comment on attachment 8540331 [details] [diff] [review]
add --enable-thread-sanitizer configure option

Review of attachment 8540331 [details] [diff] [review]:
-----------------------------------------------------------------

At some point, we may just want to replace all those options with --enable-sanitizer=foo,bar,baz.

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/a5923764a615