[Backport][GR-60108] TRegex: NFA generator improvements. #101
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.
This PR backports:
Conflicts: There were conflicts:
resolution: accept all new tests
why: extra import in jdk21 version: java.util.concurrent.TimeUnit;
resolution: move the block, keeping the needed TimeUnit import.
why: extra change in mainline:
commit oracle/graal@2fd3c55
Author: Jirka Marsik [email protected]
Date: Mon Dec 2 18:08:31 2024 +0100
Implement the JS regexp-modifiers proposal in TRegex
resolution: combine extra syntaxError parameter and direct flags field usage
why: missing code block introduced on mainline:
commit oracle/graal@2fd3c55
Author: Jirka Marsik [email protected]
Date: Mon Dec 2 18:08:31 2024 +0100
Implement the JS regexp-modifiers proposal in TRegex
resolution: accept local version
Please note. With an additional commit I disabled three tests. Reason of the fails are changes on mailine that was not backported:
The next TRegex change to backport (depends on this one): [GR-60222] TRegex: multiple fixes for look-behind merging across main expression boundary.
This PR is a part of the Oracle GraalVM for JDK 21.0.7 backports and fixes