-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Make runtime-testsuite tests run in parallel #3423
Closed
Closed
Changes from all commits
Commits
Show all changes
78 commits
Select commit
Hold shift + click to select a range
18f2471
[C++] Upstream various performance, build, and best practice fixes
jcking 709dcfb
Add .clang-format file to ensure consistent style of C++
jcking a8d8280
[C++] Bump minimum required version to C++17
jcking f711e32
Fix TSAN warnings related to dfa::DFA::s0
jcking bd37cf0
Get rid of reflection in Go.stg, fixes #2278
KvanTTT 0129cd4
Merge pull request #3311 from jcking/cpp-go-runtime-thread-safety
parrt ec144ca
Merge pull request #3327 from jcking/cpp-string-copy
parrt 53acf56
Update case-insensitive-lexing.md
tianshuang aeaa445
Merge pull request #3355 from tianshuang/patch-1
ericvergnaud 92be1ba
[C++] Cleanup ATNDeserializer interface
jcking 7fb7373
Merge branch 'master' into cpp-guid-cache
jcking d3cd381
Improve C# runtime tests performance: compile C# runtime once per ses…
KvanTTT 4695151
Get rid of dealing with files during C# runtime test running
KvanTTT 90093c8
C# runtime tests: don't write useless output to console
KvanTTT eaed457
JavaScript runtime tests: get rid of package installing using npm, us…
KvanTTT 67e34fb
Go runtime tests: get rid of useless copying of runtime files on ever…
KvanTTT 1493859
Merge pull request #3326 from jcking/cpp-guid-cache
parrt 7832bb2
Merge branch 'master' into cpp17
jcking 7a87cdc
Go runtime tests: fix working with the latest version of GO, use GORO…
KvanTTT d49facf
Go runtime tests: cache go.mod file, don't run go mod command every test
KvanTTT e811330
Merge pull request #3365 from KvanTTT/csharp-runtime-tests-performance
ericvergnaud 024e99e
Merge pull request #3335 from jcking/cpp17
parrt 12e362b
C# runtime tests: fix running on non-Windows OS
KvanTTT 0ee7f45
Fix reported memory leak
jcking 710f44e
[C++] Switch from NOEXCEPT to noexcept
jcking 652546f
Fix Dart linter errors under strict mode
jcking 0d6263a
[C++] Modernize Vocabulary using std::string_view
jcking 24d9128
[Java] Fix complaints from ErrorProne static analysis
jcking a5dfd61
Merge pull request #3380 from jcking/java-lint
parrt 3a5f112
Update JavaScript like https://github.com/antlr/antlr4/pull/3380
parrt 1d52c9b
[Java] Fix remaining complaints from ErrorProne static analysis
jcking 20c7ebf
Merge pull request #3383 from jcking/java-lint
parrt 7da07c7
Merge pull request #3376 from jcking/dart-lint
parrt f0ebf53
Go runtime tests: fix running on non-Windows OS
KvanTTT 5a23e37
Mark initializeRuntime methods with synchronized to make them multi-t…
KvanTTT d3c2253
Merge pull request #3374 from jcking/cpp-memory-leak
parrt 489de64
Merge pull request #3377 from jcking/cpp-noexcept
parrt 2f834a1
[C++] Partially undo commit a8d828016e54adfc403326d8e56d93e7c75c27fc
jcking 72e0627
Merge pull request #3393 from jcking/cpp-implicit-capture
parrt 7b2b378
[C++] Fix ambiguous ANTLRInputStream constructor overload
jcking 261769f
Merge pull request #3394 from jcking/cpp-string-view
parrt 5a798ee
update maven plug-in versions for antlr4 maven plug-in
parrt e603b73
update maven plug-in versions
parrt 34e445a
update maven plug-in versions
parrt 3801358
Merge pull request #3386 from KvanTTT/csharp-go-runtime-tests-fixes
parrt 196067d
Merge branch 'master' into update-maven-plugin-versions
parrt f45e07c
for some reason `java.*` had to become `java.**` in the mvn build w…
parrt 3d819fa
update scripts to also use python3.** meditation
parrt 7ae32c9
tweak doc for testing individual targets.
parrt 66d6679
Merge pull request #3396 from parrt/update-maven-plugin-versions
parrt 70b2edc
[C++] Switch to std::any and deprecate antlrcpp::Any
jcking 3cf70fc
Merge pull request #3395 from jcking/cpp-any
parrt 4318037
Merge pull request #3337 from jcking/clang-format
parrt ceac269
Merge pull request #3378 from jcking/cpp-vocab-string-view
parrt e8122ca
use java.** not java.* etc... in other test rigs.
parrt afb0b90
tweak antlr4 maven plugin takari thing to see if it builds
parrt e63ecf6
use maven 3.8 in github actions setup.
parrt e472a97
don't require maven 3.8 to use antlr4 maven plugin on grammars. Wasn…
parrt 14916ba
ugh. be more specific with github actions for maven version.
parrt 82c4417
1.6 -> 1.7 in our test rig compilation. tweak doc.
parrt 7e50fcf
[C++] Cleanup EMPTY_VOCABULARY uses and remove SingleWriteMultipleRea…
jcking 2349fb2
[C++] Implement standalone Unicode encoding and decoding handling
jcking bc61e31
Fixed C++ runtime demo
mike-lischke 0fd085b
Merge pull request #3402 from mike-lischke/master
parrt 43fb4c2
Merge pull request #3398 from jcking/cpp-cleanup
parrt 27ed0e9
Merge branch 'master' into cpp-vocab-cleanup
jcking 6f8d648
[C++] Optimize ATN deserialization
jcking 01a3280
Merge pull request #3400 from jcking/cpp-vocab-cleanup
parrt a850e96
Merge pull request #3401 from jcking/c-runtime
parrt 5d6a782
Merge pull request #3353 from KvanTTT/go-improvements
parrt 371ec7f
[C++] Clean up antlr4::misc::Interval and antlr4::dfa
jcking dd623ec
Merge pull request #3409 from jcking/cpp-optimize
parrt 215a188
[Go] Improve BitSet implementation
jcking 6b83a20
Merge pull request #3421 from jcking/bitset
parrt 8e9c2d0
Update README.md
parrt 329a584
[Go] Make BaseGoTest::initializeRuntime static
jcking ed31d62
Merge pull request #3422 from jcking/go-test-speed
parrt 40528ef
Make runtime-testsuite tests run in parallel
jcking File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
--- | ||
# This section defines defaults for all languages. Currently we derive ANTLR style from LLVM. | ||
BasedOnStyle: LLVM | ||
# Only use clang-format for C++ for now. | ||
DisableFormat: true | ||
|
||
--- | ||
# This section configures C++ formatting. | ||
Language: Cpp | ||
DisableFormat: false | ||
Standard: c++17 | ||
# Prevent clang-format from attempting to pick the alignment and always use right alignment. | ||
DerivePointerAlignment: false | ||
# ANTLR existing style is to right align pointers and references. | ||
PointerAlignment: Right | ||
ReferenceAlignment: Right | ||
# Some of ANTLR existing code is longer than the default 80, so use 100 for now. | ||
ColumnLimit: 100 | ||
# Historically ANTLR has used indentation within namespaces, so replicate it. | ||
NamespaceIndentation: Inner |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I recommend waiting for the merging of that request #3407 There are some troubles with JDK updating.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost ready.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like we are going to be only java 8 for this next release because there's no way to use java 11 and generate java 8 class files apparently, at least reliably.