Skip to content

Commit

Permalink
Bug 1432298 - Make sm-fuzzing build ready for libfuzzer. r=sfink
Browse files Browse the repository at this point in the history
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.

MozReview-Commit-ID: FdhCIFdUore

--HG--
extra : rebase_source : c41bda01cb42f2ef0cd5a1675d88bdb55d9dc8c9
  • Loading branch information
choller committed Jan 22, 2018
1 parent c9f2884 commit 9b23854
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 2 deletions.
4 changes: 4 additions & 0 deletions config/check_vanilla_allocations.py
Expand Up @@ -139,6 +139,10 @@ def main():
if "_memory_" in filename:
continue

# Ignore the fuzzing code imported from m-c
if "Fuzzer" in filename:
continue

fn = m.group(2)
if filename == 'jsutil.o':
jsutil_cpp.add(fn)
Expand Down
2 changes: 1 addition & 1 deletion js/src/devtools/automation/autospider.py
Expand Up @@ -276,7 +276,7 @@ def ensure_dir_exists(name, clobber=True, creation_marker_filename="CREATED-BY-A
CONFIGURE_ARGS += ' --target=x86_64-pc-mingw32 --host=x86_64-pc-mingw32'

if platform.system() == 'Linux':
CONFIGURE_ARGS += ' --enable-stdcxx-compat'
CONFIGURE_ARGS = '--enable-stdcxx-compat ' + CONFIGURE_ARGS

# Timeouts.
ACTIVE_PROCESSES = set()
Expand Down
2 changes: 1 addition & 1 deletion js/src/devtools/automation/variants/fuzzing
@@ -1,5 +1,5 @@
{
"configure-args": "--enable-fuzzing --enable-debug-symbols='-gline-tables-only -gdwarf-2' --disable-jemalloc --enable-address-sanitizer --enable-ctypes --enable-nspr-build",
"configure-args": "--enable-fuzzing --enable-debug-symbols='-gline-tables-only -gdwarf-2' --disable-jemalloc --disable-stdcxx-compat --enable-address-sanitizer --enable-ctypes --enable-nspr-build",
"optimize": true,
"debug": false,
"compiler": "clang",
Expand Down
4 changes: 4 additions & 0 deletions taskcluster/ci/spidermonkey/linux.yml
Expand Up @@ -169,3 +169,7 @@ sm-fuzzing-linux64/opt:
symbol: SM(f)
run:
spidermonkey-variant: fuzzing
toolchains:
- linux64-clang-6-pre
- linux64-gcc
- linux64-rust

0 comments on commit 9b23854

Please sign in to comment.