1432298 - Switch sm-fuzzing build to linux64-clang-6-pre

Status: RESOLVED FIXED

(Core :: JavaScript Engine, enhancement)

Description

[Christian Holler (:decoder)]

The sm-fuzzing build (SpiderMonkey with --enable-fuzzing and ASan) is still using clang 3.9 for its build, which breaks once bug 1431090 lands (because clang 3.9 does not support the trace-pc coverage flag).

Since we already have the linux64-clang-6-pre toolchain build that we use for other fuzzing builds, the sm-fuzzing build should be switched to that version as well.

Comment 1

[Christian Holler (:decoder)]

Attachment: Bug 1432298 - Make sm-fuzzing build ready for libfuzzer.

This patch disables the stdcxx-compat check for the sm-fuzzing build which
requires patching autospider as well. Furthermore, it switches the build
to linux64-clang-6-pre because the older clang 3.9 does not support trace-pc
instrumentation. Finally, it excludes fuzzing parts from the vanilla allocation
check.

Review commit: https://reviewboard.mozilla.org/r/214754/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/214754/

Comment 2

[Steve Fink [:sfink] [:s:]]

Comment on attachment 8944591 [details]
Bug 1432298 - Make sm-fuzzing build ready for libfuzzer.

https://reviewboard.mozilla.org/r/214754/#review220408

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/43f1c41b3b2e
Make sm-fuzzing build ready for libfuzzer. r=sfink

Comment 4

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/43f1c41b3b2e