Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add tests #34

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Add tests #34

wants to merge 18 commits into from

Conversation

mzbik
Copy link
Contributor

@mzbik mzbik commented Aug 22, 2020

Ben - this PR does:

  1. Minimal change to CMakeLists.txt to enable tests being built. It adds COMPILE_TEST as a variable to enable/disable compilation of tests
  2. All test CMake-related stuff is in a subdir CMakeTests
  3. This builds unittests, cctest, and cppgc_unittests all of which pass
  4. Requires no other PR

mzbik added 18 commits July 31, 2020 12:18
…BUILD.gn)

2. Small change to unify the disable-exception-defines
Adds configs back. Way simpler to handle common sets of built parameters.
Note: to run successfully, you'll need to add --timeout=2000 to the cctest invocation (via tools/run-tests.py) as well as increase the default macos pthread stack size (the compiler is now using more space).
…n terms of linker search rules for resolving symbols in libraries. Prior to this change, v8_base_without_compiler referenced v8_inspector AND v8_inspector referenced v8_base_without_compiler. GNU ld would do a search and not resolve symbols. Including the inspector object files as part of v8_base_without_compiler addresses this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant