Skip to content
This repository was archived by the owner on Nov 24, 2022. It is now read-only.

[WIP] Add nofib testing - #640

Draft
gkaracha wants to merge 90 commits into
masterfrom
wip-nofib
Draft

gkaracha wants to merge 90 commits into
masterfrom
wip-nofib

Conversation

@gkaracha

@gkaracha gkaracha commented May 7, 2020 •

Copy link
Copy Markdown
Member

Closes #639.

Set up the nofib directory:

  • Gather nofib main programs and their arguments from the upstream nofib repo.
  • Flatten the structure (only two levels) and fix/remove symlinks.
  • Remove tests that don't apply to asterius yet and other redundant files.
  • Replace local Makefiles with explicit FAST_OPTS, NORM_OPTS and SLOW_OPTS input files.

After that move on to CI-related tasks:

  • Set up the native/asterius comparison and set it up as a standalone CI job.
  • Generate V8 CPU/heap profiler report and serve them as CI artifacts, so we can examine the bottlenecks visually.

@TerrorJack

TerrorJack commented May 17, 2020 •

Copy link
Copy Markdown
Contributor

Additional notes:

  • We'll likely need to evaluate the influence of different link-time options (e.g. enabling tail calls, yolo mode, different gc thresholds), since they can have a significant impact on the overall clock time. Also, certain node V8 command-line options (use node --v8-options to get a complete list; will discuss which options to test later)
  • For getting a flame graph from V8 profiling reports, I tried a few tools and 0x seems to be pleasant to use. Use the --verbose-err link-time option to ensure wasm function names are visible in the output flame graph.

@gkaracha gkaracha mentioned this pull request May 20, 2020
@gkaracha
gkaracha force-pushed the wip-nofib branch 2 times, most recently from c824a44 to 1afe6bf Compare May 27, 2020 10:34
@TerrorJack
TerrorJack force-pushed the wip-nofib branch 2 times, most recently from b18c27a to d639214 Compare June 3, 2020 01:36
@TerrorJack
TerrorJack force-pushed the wip-nofib branch 2 times, most recently from 3076b34 to b4659f5 Compare June 17, 2020 09:12
@TerrorJack
TerrorJack force-pushed the wip-nofib branch 3 times, most recently from 6a01913 to b97c53f Compare June 23, 2020 11:35
@TerrorJack
TerrorJack force-pushed the wip-nofib branch 3 times, most recently from 2d71142 to 7da2f6e Compare July 7, 2020 11:16
@gkaracha gkaracha changed the title Add nofib testing [WIP] Add nofib testing Jul 13, 2020

This branch has not been deployed

No deployments
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add nofib performance testing

3 participants