]> granicus.if.org Git - re2c/log
re2c
7 years agoAdded test for '--posix-captures': implicit grouping is forbidden.
Ulya Trofimovich [Sat, 4 Mar 2017 11:58:55 +0000 (11:58 +0000)]
Added test for '--posix-captures': implicit grouping is forbidden.

RE2C has named definitions, which allow to override operator precedence
without grouping parenthesis. However, in POSIX all parenthesis are
capturing: allowing some of them to be non-capturing would affect
disambiguation and might result in a different parse.

For now RE2C uses a crude criterion to determine if the given named
definition needs to be explicitely wrapped in parenthesis: if the topmost
constructor of the subexpression is either alternative, or catenation, or
another reference to another named definition, then wrapping is required.
This is an overapproximation, because it doesn't take the surrounding context
into account (e.g. catenation inside of an alternative doesn't need wrapping,
but catenation inside of an iteration does).

7 years agoAdded tests for POSIX captures.
Ulya Trofimovich [Fri, 3 Mar 2017 23:21:38 +0000 (23:21 +0000)]
Added tests for POSIX captures.

Tests are adopted from Glenn Fowler's testregex suite and its derivative
regex-posix-unittest by Christopher Kuklewicz.

Irrelevant tests have been dropped:

    - tests for specific POSIX features not supported by RE2C, like
      begin/end markers '$' and '^', backreferences, predefined character
      classes (most of them come from 'basic' subdirectory)

    - all 'leftassoc' tests: they are exact copies of 'rightassoc' tests,
      differ only in match results and are supposed to fail

    - all tests from 'categorize' that are supposed to fail

I also added a few tests of my own (in the 'other' subdirectory).

Tests are arranged as a bunch of '.dat' files (each file contains lines of
the form '<regexp> <string> <match>' or blank lines) and two scripts.

    - '__run.sh' runs all tests (generates temporary '.re' file with the
      given regexp, compiles it, runs against the given string and checks
      that the result coincides with the given result)

    - '__gen.sh' regenerates '.re' files for regular RE2C test suite (the
      generated files do not contain any code, just the bare regexp)

All regexps are pre-patched to exclude '\x00' (terminating NULL) character
from subregexps like '.' or '[^a]: otherwise RE2C-generated programs would
greedily consume all characters and fail. Note that if only '.' was present
in the original tests, we might have used '\n' as terminator, but '[^a]'
includes newline. Also '()' subregexp was substituted with '("")'.

7 years ago'--skeleton --posix-captures': use correct capture index when checking tags.
Ulya Trofimovich [Fri, 3 Mar 2017 23:09:19 +0000 (23:09 +0000)]
'--skeleton --posix-captures': use correct capture index when checking tags.

Instead of skipping fixed tags, skip orbit tags. For each orbit tag there is
a tag fixed on it, so if we check all fixed tags we'll surely check all orbit
tags. Anyway, with '--skeleton' all fixed tags correspond to orbit tags: other
fixed tags are forbidden with generic API (even zero-offset ones for now).

7 years agoRemove dead 'copy' commands.
Ulya Trofimovich [Fri, 3 Mar 2017 17:27:35 +0000 (17:27 +0000)]
Remove dead 'copy' commands.

For some reason dead code elimination pass simply ignored 'copy' commands
and removed only dead 'save' commands. Strange as it seems, this went
unnoticed until now (when one of the POSIX tests compiled into C++ code
that triggered GCC warning about use of uninitialized variable).

7 years agoFixed code generation for '--skeleton' with '--posix-captures'.
Ulya Trofimovich [Fri, 3 Mar 2017 17:06:31 +0000 (17:06 +0000)]
Fixed code generation for '--skeleton' with '--posix-captures'.

7 years agoAdded option '--posix-captures'.
Ulya Trofimovich [Fri, 3 Mar 2017 02:06:43 +0000 (02:06 +0000)]
Added option '--posix-captures'.

With this option re2c interprets all parenthetised subexpressions
as POSIX capturing groups and returns submatch according to POSIX
disambiguation rules (in variables 'yynmatch' and 'yypmatch', local
to each action).

The algorithm for disambiguation was invented by Christopher Kuklewicz.

7 years agoNow code generation can handle zero-offset fixed tags with generic API.
Ulya Trofimovich [Fri, 3 Mar 2017 01:47:03 +0000 (01:47 +0000)]
Now code generation can handle zero-offset fixed tags with generic API.

We'll need this for POSIX captures: in the prsence of orbit tag,
opening tag is needed only for disambiguation; after determinization
we can pretend that it is fixed on orbit tag with zero offset.
This will save some time and space on tag optimizations and maybe
reduce the number of tag variables.

7 years agoPrepare default tags insertion for non-sequential tag enumeration.
Ulya Trofimovich [Fri, 3 Mar 2017 01:21:53 +0000 (01:21 +0000)]
Prepare default tags insertion for non-sequential tag enumeration.

This will be needed when we add POSIX captures: opening and closing
capture tags must have sequential numbers, but the tags may be located
far from each other in the regexp tree (other tags are possible on
the way from the opening tag to the closing tag).

7 years agoDelay expansion of counted repetition until NFA construction time.
Ulya Trofimovich [Fri, 3 Mar 2017 01:15:05 +0000 (01:15 +0000)]
Delay expansion of counted repetition until NFA construction time.

This is necessary to avoid creating default tags for tags that
are duplicated by expansion of counted repetition. Default tags
should only be created if zero repetitions are acceptable; this case
is still expanded early.

7 years agoKeep the full history of tag occurences found by epsilon-closure.
Ulya Trofimovich [Fri, 3 Mar 2017 00:56:49 +0000 (00:56 +0000)]
Keep the full history of tag occurences found by epsilon-closure.

Currently this makes no difference because tag duplication in NFA
is forbidden. Anyway, we are only interested in the last occurence
of each tag on the epsilon-path to the given NFA state.

This will make difference for POSIX disambiguation strategy: we'll
have to allow tag duplication in NFA (because of counted repetition
and iteration expansion).

7 years agoMoved splitting charset and nullable rule analysis from AST to regexp.
Ulya Trofimovich [Sat, 25 Feb 2017 19:04:14 +0000 (19:04 +0000)]
Moved splitting charset and nullable rule analysis from AST to regexp.

7 years agoAdded yet another intermediate representation (after AST, before NFA).
Ulya Trofimovich [Sat, 25 Feb 2017 17:00:49 +0000 (17:00 +0000)]
Added yet another intermediate representation (after AST, before NFA).

Rationale: AST must be immutable, because it is shared among different
regexps, rules, conditions and blocks. NFA is mutable, but inconvenient
for some transformations (adding fixed and default tags, rewriting parts
of regexp, etc.). Before this commit all these transformations were done
in one pass (during convertion of AST to NFA) which made the code rather
complex. In order to add POSIX captures we'll need to add even more
complexity (tracking captures, creating tags for them, maintaining tag
order, etc.).

7 years ago'--dump-nfa': correctly print symbol ranges on NFA transitions.
Ulya Trofimovich [Fri, 24 Feb 2017 17:15:14 +0000 (17:15 +0000)]
'--dump-nfa': correctly print symbol ranges on NFA transitions.

7 years agoKeep fixed and variable tags together in one array.
Ulya Trofimovich [Fri, 24 Feb 2017 16:59:33 +0000 (16:59 +0000)]
Keep fixed and variable tags together in one array.

One must take fixed tags into account during determinization
if disambiguation policy is based on tag values.

Also, NFA construction is much easier when tags have common enumeration:
it allows to perdorm tag creation, tag fixing and adding default tags
in three distinct passes (for now they are intermixed and the code is
very complex).

7 years agoStop preserving order of tag versions when mapping TDFA states.
Ulya Trofimovich [Tue, 14 Feb 2017 14:51:47 +0000 (14:51 +0000)]
Stop preserving order of tag versions when mapping TDFA states.

We no longer need it; now that disambiguation policy is leftmost rather
than tag-centric, relative order of tag versions is irrelevant.
Tag versions have no impact on conflict resolution; any bijective
mapping is ok.

7 years agoMoved tests to subdirectory.
Ulya Trofimovich [Tue, 14 Feb 2017 09:53:24 +0000 (09:53 +0000)]
Moved tests to subdirectory.

7 years agoChanged disambiguation policy: leftmost rather than tag-centric.
Ulya Trofimovich [Mon, 13 Feb 2017 21:59:06 +0000 (21:59 +0000)]
Changed disambiguation policy: leftmost rather than tag-centric.

Instead of tracking priority for each tag version and resolving conflicting
configurations encountered by the epsilon-closure based on version priorities
(trying to minimize or maximize each tag), we now simply pick the leftmost
epsilon-path through the NFA.

NFA construction has to accommodate this policy: for example, greedy iteration
should be compiled so that always choosing the left alternative correpongs to
re-iteration, while choosing the right alternative corresponds to leaving the
iteration. The necessary adjustments have been done by earlier commit
03c65121da3f8bcb412576568cfff6cbca039470.

Now we can drop priority comparison when mapping TDFA states and stick to
simple bijection. This may add cycles in reordering commands. (Has not been
done yet.)

The rationale of switching to leftmost policy is this: with tag maximization/
minimization policy tags could influence the regexp: for example, regexp
    "a"* @p "a"*
has different meaning depending on whether tag 'p' is minimized (first iteration
is non-greedy) or maximized (first iteration is greedy). Moreover, adding
another tag with the opposite policy and higher priority would change the
meaning (suppose 'q' is minimized and earlier tags have higher priority):
    "a"* @q @p "a"*
This all makes regexps somewhat fragile, since adding just one tag may change
the whole regexp (and in the presence of other tags, it may change parse
results).

But even worse, with tag-centric disambiguation simple (untagged) subexpressions
are inferior to tagged ones. This contradicts intuition and even POSIX standard,
where  a*(a*)  is the same as  (a*)(a*) . We don't keep to POSIX (and we can't
because of last-longest instead of first-longest semantics of iteration as shown
by the  (<|<a|<ab|<aba|abab|baba|b>|>)*  example), but even this somewhat
tag-centric standard respects non-captured parts of the regexp. Note that
Laurikari in his TRE library has to do special preprocessing of the regexp in
order to assign correct minimization/maximisation direction to each tag
depending on the context of the tag. His original tag-centric policy (open tag
is minimized and has higher priority than closing tag which is maximized,
nested tags have lower priority) is insufficient to accommodate POSIX.

7 years agoDon't trace tags in epcilon-cycles when building tagged epsilon-closure.
Ulya Trofimovich [Mon, 13 Feb 2017 21:00:49 +0000 (21:00 +0000)]
Don't trace tags in epcilon-cycles when building tagged epsilon-closure.

7 years agoIn greedy regexps first alternative must correspond to consuming path.
Ulya Trofimovich [Mon, 13 Feb 2017 15:01:12 +0000 (15:01 +0000)]
In greedy regexps first alternative must correspond to consuming path.

By convention first alternative has higher priority.
So, for example, the following must be true:
    r*     = rr* | <empty>
    r{n,m} = r{n} (r{m - n} | r{m - n - 1} | ... | r{1} | <empty>)
    r{n,}  = r{n} (rr* | <empty>)

For now we don't care about priorities: this is a preparatory step
before transition to greedy leftmost semantics for tags.

7 years agoTags that occur earlier in regular expression must have greater priority.
Ulya Trofimovich [Mon, 13 Feb 2017 12:52:32 +0000 (12:52 +0000)]
Tags that occur earlier in regular expression must have greater priority.

7 years agoAdded debug option '--dump-nfa'.
Ulya Trofimovich [Mon, 13 Feb 2017 12:37:40 +0000 (12:37 +0000)]
Added debug option '--dump-nfa'.

7 years agore2c: cleanup a few clang ++ warnings
Sergei Trofimovich [Sun, 12 Feb 2017 11:16:30 +0000 (11:16 +0000)]
re2c: cleanup a few clang ++ warnings

Found those as:

    $ make CXXFLAGS="-Weverything -Werror -Wno-gnu-statement-expression -Wno-nested-anon-types"

- cleaned struct<->class declaration mismatch
- 'static' around local function
- 'RE2C_GXX_ATTRIBUTE ((noreturn))' around functions known to fail

Signed-off-by: Sergei Trofimovich <slyfox@gentoo.org>
7 years agoPack lookahead tags together with tag versions in TDFA state.
Ulya Trofimovich [Thu, 9 Feb 2017 16:01:51 +0000 (16:01 +0000)]
Pack lookahead tags together with tag versions in TDFA state.

This packing is possible because we are never interested in version
of the tag if the given item has this tag in lookahead set.

7 years agoUse 'normal form' of version matrix to test if states are mappable.
Ulya Trofimovich [Thu, 9 Feb 2017 13:10:31 +0000 (13:10 +0000)]
Use 'normal form' of version matrix to test if states are mappable.

Normal form is calculated for every new state before searching for
a mappable counterpart. All versions are renumbered to occupy consequent
numbers starting from one; version order is preserved.

7 years agoAdded tests that fail if mapping of TDFA states ignores version order.
Ulya Trofimovich [Thu, 9 Feb 2017 13:01:26 +0000 (13:01 +0000)]
Added tests that fail if mapping of TDFA states ignores version order.

These tests result in incorrect automaton unless mapping preserves
relative order of versions in TDFA states (respects version priority).

7 years agoStop supporting non-bijective mappings of TDFA states.
Ulya Trofimovich [Thu, 9 Feb 2017 12:58:43 +0000 (12:58 +0000)]
Stop supporting non-bijective mappings of TDFA states.

They are not very helpful, yet add much complexity to the mapping step.
This commit removes option '--non-bijective-mappings'.

7 years agoAssert that operands of 'copy' command are different.
Ulya Trofimovich [Wed, 8 Feb 2017 17:03:57 +0000 (17:03 +0000)]
Assert that operands of 'copy' command are different.

7 years ago'--dump-dfa-raw': better formatting of item's tags.
Ulya Trofimovich [Wed, 1 Feb 2017 13:18:13 +0000 (13:18 +0000)]
'--dump-dfa-raw': better formatting of item's tags.

7 years ago'--dump-dfa-raw': show proper shadowed items with updated tag versions.
Ulya Trofimovich [Wed, 1 Feb 2017 09:47:01 +0000 (09:47 +0000)]
'--dump-dfa-raw': show proper shadowed items with updated tag versions.

7 years agoAllocate final tag versions on the go instead of preallocating versions [1 .. N].
Ulya Trofimovich [Tue, 31 Jan 2017 18:53:57 +0000 (18:53 +0000)]
Allocate final tag versions on the go instead of preallocating versions [1 .. N].

One cannot rely the fact that final versions are [1 .. N] anyway (they are renamed
during tag optimization), so this is not convenient as it seems.

Now versions start from 1, which looks better (more logical) on the pictures.

7 years ago'--dump-dfa-raw': show shadowed NFA sub-states.
Ulya Trofimovich [Tue, 31 Jan 2017 09:50:58 +0000 (09:50 +0000)]
'--dump-dfa-raw': show shadowed NFA sub-states.

7 years agoFixed yet another case of disordered tags update and cursor increment.
Ulya Trofimovich [Wed, 25 Jan 2017 17:50:45 +0000 (17:50 +0000)]
Fixed yet another case of disordered tags update and cursor increment.

/* note [tag hoisting, skip hoisting and tunneling]
 *
 * Tag hoisting is simple: if all transitions have the same commands,
 * they can be hoisted out of conditional branches.
 *
 * Skip hoisting is only relevant with '--eager-skip' option.
 * Normally this option is off and skip is lazy: it happens after
 * transition to the next state, if this state is consuming.
 * However, with '--eager-skip' skip happens before transition to the next
 * state. Different transitions may disagree: some of them go to consuming
 * states, others don't. If they agree, skip can be hoisted (just like tags).
 *
 * '--eager-skip' makes tag hoisting more complicated, because now we have
 * to care about the type of automaton: lookahead TDFAs must skip after
 * writing tags, while non-lookahead TDFAs must skip before writing tags.
 * Therefore skip hoising cannot be done without tag hoisting in lookahead
 * TDFAs, and vice versa with non-lookahead TDFAs.
 * (Note that '--eager-skip' is implied by '--no-lookahead').
 *
 * Tunneling splits base states in two parts: head and body. Body has all
 * the conditional branches (transitions on symbols), while head has just
 * one unconditional jump to body.
 *
 * Normally tag hoisting should go before tunneling: hoisting may add new
 * candidates to be merged by tunneling. However, with '--eager-skip' tag
 * hoisting is interwined with skip hoisting, and the latter needs to know
 * which states are consuming. This is not possible if tunneling is still
 * to be done, because it may turn consuming states into non-consuming ones.
 * Another option is to disallow splitting states with non-hoisted skip
 * in the presence of '--eager-skip' (this way skip hoisting wouldn't need
 * to know tunneling results), but it's much worse for tunneling.
 */

Found by slyfox's fuzzer. :)

7 years agoDon't loose 'YYBACKUPTAG' command in case of equal mapped versions.
Ulya Trofimovich [Mon, 23 Jan 2017 22:28:16 +0000 (22:28 +0000)]
Don't loose 'YYBACKUPTAG' command in case of equal mapped versions.

Found by slyfox's fuzzer. :)

7 years agocorrect some minor manpage typos
jcfp [Mon, 23 Jan 2017 19:29:07 +0000 (20:29 +0100)]
correct some minor manpage typos

these showed up in lintian during a maintenance update of the debian re2c package

7 years agoUpdate tags after reading character in TDFAs without lookahead.
Ulya Trofimovich [Mon, 23 Jan 2017 21:02:24 +0000 (21:02 +0000)]
Update tags after reading character in TDFAs without lookahead.

Fixed forgotten special case when tags and cursor increment
are hoisted out of conditional branches.

Found by slyfox's fuzzer. :)

7 years agoFixed memory leak.
Ulya Trofimovich [Mon, 23 Jan 2017 18:09:05 +0000 (18:09 +0000)]
Fixed memory leak.

Found by valgrind:

 1,177 bytes in 1 blocks are definitely lost in loss record 1 of 1
    at 0x4C2AE40: operator new[](unsigned long) (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
    by 0x42153F: re2c::cfg_t::compact(re2c::cfg_t const&, int*) (compact.cc:12)
    by 0x422088: re2c::optimize_tags(re2c::dfa_t&) (optimize.cc:18)
    by 0x42DE3A: re2c::compile(re2c::spec_t const&, re2c::Output&) (compile.cc:82)
    by 0x446742: re2c::parse(re2c::Scanner&, re2c::Output&) (parser.ypp:484)
    by 0x402DC0: main (main.cc:38)

7 years agoIterate CFG nodes in postorder to speed up liveness analysis.
Ulya Trofimovich [Mon, 23 Jan 2017 18:01:49 +0000 (18:01 +0000)]
Iterate CFG nodes in postorder to speed up liveness analysis.

Deep-first search postorder is the best possible order for solving
control-flow equations for liveness analysis.

7 years agoUse the same tag version for all transitions from the given state.
Ulya Trofimovich [Mon, 23 Jan 2017 17:42:37 +0000 (17:42 +0000)]
Use the same tag version for all transitions from the given state.

This reduces the overall number of versions greatly, which is crucial
for the space and time efficiency of tag liveness analysis.

Of course, bottom and normal transitions still have different versions.

Differents tags on the same transition also have different versions.
It is not incorrect to give them the same version: all tags updated by
one and the same transition are either set to bottom, or to current
input position (which is the same for all tags on the given transition).
But this would create many states with a very special "shape" that do
not map to other states. In fact, this would cause exponential blowup
in the number of states (even in simple cases).

7 years agoCompact tag versions before doing liveness analysis.
Ulya Trofimovich [Mon, 23 Jan 2017 16:38:04 +0000 (16:38 +0000)]
Compact tag versions before doing liveness analysis.

After TDFA construction tag versions occupy non-contiguous range
of natural numbers: there may be "holes" between versions. This wastes
space (and time) spent on liveness analysis and subsequent optimization
passes. This commit adds a simple pass that renumbers versions so that
they occupy contiguios range.

7 years agoUpdated parser for bison-3.0.4.
Ulya Trofimovich [Mon, 23 Jan 2017 16:19:24 +0000 (16:19 +0000)]
Updated parser for bison-3.0.4.

7 years agoAdded option '--no-lookahead'.
Ulya Trofimovich [Wed, 18 Jan 2017 13:58:48 +0000 (13:58 +0000)]
Added option '--no-lookahead'.

This option makes re2c generate TDFA that does not exploit lookahead
symbol when making tagged transitions.

7 years agoAdded option '--eager-skip'.
Ulya Trofimovich [Tue, 17 Jan 2017 12:36:43 +0000 (12:36 +0000)]
Added option '--eager-skip'.

This option allows to control program points in which cursor is
advanced to the next input character. Normally it happens after moving
to the next state (lazyly). With '--eager-skip' cursor is advanced
before moving to the next state (eagerly).

Eager mode presents some difficulties compared to the lazy mode:
transitions may disagree on whether advance cursor or not. Some of
them may go to accepting states and should not advance cursor, while
others go normal states and should advance cursor. In such cases
advancement must happen on a per-transition basis. If all transitions
agree on the matter, advancement can be hoisted out and performed on
a per-state basis, just as it happens in lazy mode.

7 years agoReuse mode: always reparse rules and restore options of the rules block.
Ulya Trofimovich [Sun, 8 Jan 2017 10:34:04 +0000 (10:34 +0000)]
Reuse mode: always reparse rules and restore options of the rules block.

7 years agoConsistently use current block's options for code generation.
Ulya Trofimovich [Sun, 1 Jan 2017 18:47:06 +0000 (18:47 +0000)]
Consistently use current block's options for code generation.

7 years agoFold output expressions in a separate pass over the output.
Ulya Trofimovich [Fri, 30 Dec 2016 17:15:16 +0000 (17:15 +0000)]
Fold output expressions in a separate pass over the output.

re2c folds sequences of simple statements to complex tatements, e.g.:
    ++YYCURSOR;
    yych = *YYCURSOR;
is folded to:
    yych = *++YYCURSOR;

It is hard to fold expressions on the fly (at the time they are generated),
because one needs to forsee all the cases in advance. Some cases cannot be
anticipated (e.g. if substatements belong to different states, which is often
the case), so many statements remain unfolded.

This commit separates code generation and folding by delaying output of
skip, peek and backup primitives. Folding now is done in a separate pass
when all the substatements are already known.

8 years agoMemorize options for each block and use them for delayed code generation.
Ulya Trofimovich [Thu, 29 Dec 2016 17:07:52 +0000 (17:07 +0000)]
Memorize options for each block and use them for delayed code generation.

Option scope is very poorly defined in re2c. Most options should have block
scope, but some have global scope (because they belong to global directives
like '/*!types:re2c ... */', '/*!tags:re2c ... */' or other global things
like generation date, etc.).

For now, options are applied immediately as they are parsed.
Code generation just reads the most recent state of options, so the output
depends not only on the location in source code when the option is defined,
but also on program points in which re2c generates and outputs code.
Generation and output may happen at different times, so they may use
different options. This is all very bad.

This commit is the first attempt to introduce scoped options: re2c now
makes a snapshot of options at the end of each block and uses this snapshot
for delayed code generation for this block (unless the given option must be
global).

This is not perfect (immediate code generation still uses 'seen-so-far'
options instead of block options), but at least it allows to delay code
generation for non-global things without loosing options.

8 years agoDelay code generation for condition dispatch.
Ulya Trofimovich [Tue, 27 Dec 2016 21:04:09 +0000 (21:04 +0000)]
Delay code generation for condition dispatch.

8 years agoFinally separated parsing, consistensy checking and after-parse transformation.
Ulya Trofimovich [Tue, 27 Dec 2016 17:00:09 +0000 (17:00 +0000)]
Finally separated parsing, consistensy checking and after-parse transformation.

Updates in test results are caused by the changed order of condition
compilation, see commit b7bbbf471778b487d0717c85a5e586b595e495ec.
Changes do not affect code generation, only the error messages.

8 years agoEmit conditions ordered by their number rather than lexicograhically.
Ulya Trofimovich [Sun, 25 Dec 2016 18:13:43 +0000 (18:13 +0000)]
Emit conditions ordered by their number rather than lexicograhically.

Each condition has a unique name and unique number. Numbers are assigned
to conditions in the order they appear in source file. This order must
be preserved since some users rely on it (user code makes implicit
assumptions about condition order, see '-Wcondition-order').

However, once the order is preserved, the automata code can be emitted
in any order (user code can only address conditions by their start label,
and it can be anywhere in the output file).

This commit changes the order of automata: now they are emitted according
to their numbers, not names.

Test results have changed in cases when lexicographical order does not
coinside with number order. The changes touch some important files and
are not easy to verify. However, rearranging conditions in the generated
code back in lexicographical order makes the differences trivial (labels,
'YYDEBUG' parameter, 'YYFILL' labels, 'YYSETSTATE' parameter --- in other
words, everything that depends on state numbering). It is easy to see
that the structure of the code is not changed.

8 years agoReuse 'Code' data struct for setup code.
Ulya Trofimovich [Sat, 24 Dec 2016 12:00:59 +0000 (12:00 +0000)]
Reuse 'Code' data struct for setup code.

8 years agoReduced one argument to function as it is a part of another argument.
Ulya Trofimovich [Sat, 24 Dec 2016 11:20:48 +0000 (11:20 +0000)]
Reduced one argument to function as it is a part of another argument.

8 years agoParser: started gathering all error messages in one place.
Ulya Trofimovich [Sat, 24 Dec 2016 10:40:08 +0000 (10:40 +0000)]
Parser: started gathering all error messages in one place.

8 years agoKeep startup code '<>' separately from other rules.
Ulya Trofimovich [Fri, 23 Dec 2016 16:53:31 +0000 (16:53 +0000)]
Keep startup code '<>' separately from other rules.

8 years agoKeep '*' rules separately from other rules.
Ulya Trofimovich [Fri, 23 Dec 2016 16:10:50 +0000 (16:10 +0000)]
Keep '*' rules separately from other rules.

8 years agoSetup code for rules belongs in DFA scope, not the entire block scope.
Ulya Trofimovich [Fri, 23 Dec 2016 15:55:01 +0000 (15:55 +0000)]
Setup code for rules belongs in DFA scope, not the entire block scope.

8 years agoRestructured parser grammar to explicitly list different rule types.
Ulya Trofimovich [Fri, 23 Dec 2016 15:01:28 +0000 (15:01 +0000)]
Restructured parser grammar to explicitly list different rule types.

8 years agoParser grammar: separate production for code in conditional rules.
Ulya Trofimovich [Thu, 22 Dec 2016 22:11:08 +0000 (22:11 +0000)]
Parser grammar: separate production for code in conditional rules.

8 years agoPass context as parameter to 'yyparse' instead of a bunch of static variables.
Ulya Trofimovich [Thu, 22 Dec 2016 17:36:15 +0000 (17:36 +0000)]
Pass context as parameter to 'yyparse' instead of a bunch of static variables.

As a side effect of this commit a long-standing bug with setup rules
has been fixed: re2c forgot to clean up the list of setup rules together
with other data, which resulted in false errors about duplicate setup code.

8 years agoRemoved unused typedef.
Ulya Trofimovich [Thu, 22 Dec 2016 16:56:15 +0000 (16:56 +0000)]
Removed unused typedef.

8 years agoFixed memleaks (found by valgrind).
Ulya Trofimovich [Thu, 22 Dec 2016 16:53:41 +0000 (16:53 +0000)]
Fixed memleaks (found by valgrind).

8 years agoHandle all types of rules in one place (in parser).
Ulya Trofimovich [Thu, 22 Dec 2016 14:40:32 +0000 (14:40 +0000)]
Handle all types of rules in one place (in parser).

8 years agoSimplified handling of user-defined code in parser.
Ulya Trofimovich [Thu, 22 Dec 2016 13:58:05 +0000 (13:58 +0000)]
Simplified handling of user-defined code in parser.

8 years agoInlined small function with only one user.
Ulya Trofimovich [Thu, 22 Dec 2016 12:04:59 +0000 (12:04 +0000)]
Inlined small function with only one user.

8 years agoSimplified line tracking in lexer.
Ulya Trofimovich [Thu, 22 Dec 2016 11:58:48 +0000 (11:58 +0000)]
Simplified line tracking in lexer.

8 years agoSeparated compilation pass from code generation pass.
Ulya Trofimovich [Thu, 22 Dec 2016 09:54:13 +0000 (09:54 +0000)]
Separated compilation pass from code generation pass.

They might be totally disconnected in case of '-r' option and it's
more convenient to handle them separately.

8 years agoParse rules in a uniform way regardless of the '-c' option.
Ulya Trofimovich [Wed, 21 Dec 2016 23:14:17 +0000 (23:14 +0000)]
Parse rules in a uniform way regardless of the '-c' option.

8 years agoSimplified parsing of startup code '<>'.
Ulya Trofimovich [Wed, 21 Dec 2016 22:31:34 +0000 (22:31 +0000)]
Simplified parsing of startup code '<>'.

8 years agoDon't use special token type for '<!' and '<>'.
Ulya Trofimovich [Wed, 21 Dec 2016 18:44:02 +0000 (18:44 +0000)]
Don't use special token type for '<!' and '<>'.

Parse them as character sequences.

8 years agoFurther simplified default rule parsing.
Ulya Trofimovich [Wed, 21 Dec 2016 17:21:31 +0000 (17:21 +0000)]
Further simplified default rule parsing.

8 years agoSimplified handling of default rule in parser.
Ulya Trofimovich [Wed, 21 Dec 2016 17:12:09 +0000 (17:12 +0000)]
Simplified handling of default rule in parser.

8 years agoDon't loose condition if it consists of default rule only.
Ulya Trofimovich [Wed, 21 Dec 2016 11:43:22 +0000 (11:43 +0000)]
Don't loose condition if it consists of default rule only.

8 years agoSimplified '*' condition handling in parser.
Ulya Trofimovich [Tue, 20 Dec 2016 22:45:23 +0000 (22:45 +0000)]
Simplified '*' condition handling in parser.

8 years agoMoved variable from global scope to function scope.
Ulya Trofimovich [Tue, 20 Dec 2016 09:39:52 +0000 (09:39 +0000)]
Moved variable from global scope to function scope.

8 years agoMake 'Scanner' parameter to 'yyparse' instead of global variable.
Ulya Trofimovich [Mon, 19 Dec 2016 09:59:15 +0000 (09:59 +0000)]
Make 'Scanner' parameter to 'yyparse' instead of global variable.

8 years agoRemoved empty header 'globals.h' and all includes of it.
Ulya Trofimovich [Mon, 19 Dec 2016 08:45:46 +0000 (08:45 +0000)]
Removed empty header 'globals.h' and all includes of it.

8 years agoRemoved variable from global scope.
Ulya Trofimovich [Mon, 19 Dec 2016 08:42:14 +0000 (08:42 +0000)]
Removed variable from global scope.

8 years agoRemoved variable from global scope.
Ulya Trofimovich [Mon, 19 Dec 2016 08:37:46 +0000 (08:37 +0000)]
Removed variable from global scope.

8 years agoRemoved variable from global scope, simplified bitmaps.
Ulya Trofimovich [Mon, 19 Dec 2016 00:06:19 +0000 (00:06 +0000)]
Removed variable from global scope, simplified bitmaps.

8 years agoRemoved variable from global scope.
Ulya Trofimovich [Sun, 18 Dec 2016 16:37:05 +0000 (16:37 +0000)]
Removed variable from global scope.

8 years agoRemoved variable from global scope.
Ulya Trofimovich [Sun, 18 Dec 2016 14:56:10 +0000 (14:56 +0000)]
Removed variable from global scope.

8 years agoRemoved warnings from global scope.
Ulya Trofimovich [Sun, 18 Dec 2016 12:27:19 +0000 (12:27 +0000)]
Removed warnings from global scope.

8 years agoRemoved options from global scope.
Ulya Trofimovich [Sun, 18 Dec 2016 12:02:05 +0000 (12:02 +0000)]
Removed options from global scope.

8 years agoSimplified code generation for initial state.
Ulya Trofimovich [Wed, 14 Dec 2016 16:25:54 +0000 (16:25 +0000)]
Simplified code generation for initial state.

8 years agoSimplified generation of code for buffer refilling.
Ulya Trofimovich [Wed, 14 Dec 2016 15:16:01 +0000 (15:16 +0000)]
Simplified generation of code for buffer refilling.

8 years agoDon't delay reading next character until dispatch.
Ulya Trofimovich [Wed, 14 Dec 2016 14:21:28 +0000 (14:21 +0000)]
Don't delay reading next character until dispatch.

Instead of sometimes generating code like this:

    ++YYCURSOR;
    switch ((yych = *YYCURSOR)) {

re2c now always generates

    yych = *++YYCURSOR;
    switch (yych) {

There's not much difference: it's hard to say which would be better
optimized by C/C++ compiler. Most likely they would be optimized
equally well; practical experiments with GCC -O2 resulted in bit-to-bit
identical binaries.

However, dropping the first form allows to cut off a lot of tricky code
in re2c itself (a whole layer of code spread across multiple files was
tracking whether next input character has been already read or not).

Changes to the test results are multiple, but trivial; they have been
verified with skeleton.

8 years agoDon't peek next input character if all transitions go to final state.
Ulya Trofimovich [Wed, 14 Dec 2016 13:35:21 +0000 (13:35 +0000)]
Don't peek next input character if all transitions go to final state.

A similar heuristic already existed, but it was imprecise and captured
only those cases when final state happened to be next to the one in
question.

A lot of tests were updated; however changes are trivial and I verified
them with skeleton.

8 years agoRenamed option '--dfa-mapping <bijective | injective>' to '--non-bijective-mapping'.
Ulya Trofimovich [Mon, 12 Dec 2016 13:44:00 +0000 (13:44 +0000)]
Renamed option '--dfa-mapping <bijective | injective>' to '--non-bijective-mapping'.

8 years agoSearch for the best candidate in case of non-bijective mapping.
Ulya Trofimovich [Mon, 12 Dec 2016 13:16:56 +0000 (13:16 +0000)]
Search for the best candidate in case of non-bijective mapping.

For bijective mappings there is always at most one mapping candidate,
for non-bijective mappings there may be many candidates.

8 years agoRespect injective mappings when optimizing save(X), copy(Y,X) to save(Y).
Ulya Trofimovich [Fri, 9 Dec 2016 12:15:56 +0000 (12:15 +0000)]
Respect injective mappings when optimizing save(X), copy(Y,X) to save(Y).

If mapping is injective, there may be multiple copy commands per one
save command: save(X), copy(Y,X), copy(Z,X). In this case save command
must be replicated for each copy command: save(Y), save(Z).

8 years agoNow checking for tag priority violation respects injective mappings.
Ulya Trofimovich [Thu, 8 Dec 2016 17:34:12 +0000 (17:34 +0000)]
Now checking for tag priority violation respects injective mappings.

8 years agoFixed atrocious typo that unconditionally enabled injective mapping.
Ulya Trofimovich [Thu, 8 Dec 2016 13:31:28 +0000 (13:31 +0000)]
Fixed atrocious typo that unconditionally enabled injective mapping.

However, injectivity never showed up because every injective mapping
(as of now) is erroneously considered to violate tag priority.

8 years agoAdded test: exponential blowup if bottom is treated as special version.
Ulya Trofimovich [Wed, 7 Dec 2016 17:09:14 +0000 (17:09 +0000)]
Added test: exponential blowup if bottom is treated as special version.

One tempting way to track default value of tags is to forbid mapping
bottom (default) to non-bottom versions. This way we don't need to
explicitely save bottom to variable; bottom value is encoded in state
(once a tag becomes bottom, it will never become non-bottom again).

However, this may exponentially blow up the number of states.

8 years agoCleaned up determinization.
Ulya Trofimovich [Wed, 7 Dec 2016 15:08:58 +0000 (15:08 +0000)]
Cleaned up determinization.

Moved creation of new DFA states and finalizers to a separate routine,
together with building tag commands.

8 years agoUntwined determinization and '-Wnondeterministic-tags' analysis.
Ulya Trofimovich [Tue, 6 Dec 2016 13:35:07 +0000 (13:35 +0000)]
Untwined determinization and '-Wnondeterministic-tags' analysis.

Now '-Wnondeterministic-tags' analysis is done right after determinization,
while we still have all the kernels available. Instead of comparing
transition tags (which are available in closure items, but not in kernels)
we now compare tag versions: maximum number of distinct versions per tag is
the desired degree of nondeterminism. Kernel items that belong to another
rule do not count (all tags that don't belong to a particular rule should
have initial versions in all items belonging to this rule).

Added test with an example of pathological regexp: counted repetition
cannot be folded efficiently in DFA, and combined with nondeterministic
tags it results in arbitrary long chains of reordering commands.
Length of command choin grows linearly with the number of repetitions
(as DFA itself does).

8 years agoDebug '--dump-adfa' option: drop 'YYFILL' in move states, it's fictive.
Ulya Trofimovich [Mon, 5 Dec 2016 22:19:37 +0000 (22:19 +0000)]
Debug '--dump-adfa' option: drop 'YYFILL' in move states, it's fictive.

8 years agoAdded debug option '--dump-dfa-raw'.
Ulya Trofimovich [Mon, 5 Dec 2016 17:32:41 +0000 (17:32 +0000)]
Added debug option '--dump-dfa-raw'.

It dumps DFA at the very early stage of construction, expanding underlying
state kernels with NFA substates and correspondings sets of tag versions.

8 years agoAdded option '--dfa-mapping <bijective | injective>'.
Ulya Trofimovich [Sun, 4 Dec 2016 14:38:53 +0000 (14:38 +0000)]
Added option '--dfa-mapping <bijective | injective>'.

This option controls how tag versions in DFA states are mapped to each
other during determinization. By default mapping is 'bijective', however
if it is set to 'injective' the new state must be less versatile than
the old one.

8 years agoDebug '--dump-adfa' option: correctly display range lower bound.
Ulya Trofimovich [Sun, 4 Dec 2016 12:16:19 +0000 (12:16 +0000)]
Debug '--dump-adfa' option: correctly display range lower bound.

8 years agoDrop hoisted tags in 'base' states duplicated by tunneling.
Ulya Trofimovich [Sun, 4 Dec 2016 12:00:31 +0000 (12:00 +0000)]
Drop hoisted tags in 'base' states duplicated by tunneling.

Tunneling optimization finds certain states and splits them into
two parts: First part consumes a character and unconditionally moves
to the second part. Second part does not consume characters, but
contains a switch on character value.

Second part is a 'base switch', which may be utilized by many other
states. So if the first part has hoisted tags, they should be dropped
in the second part (the aforementioned many other states may have hoisted
tags of there own or no hoisted tags at all).

Found by slyfox's fuzzer. :)

Yet unable to construct a test case that reveals the error: the original
test case found by fuzzer is not breaking anymore after recent commit
44045203c211531cc3e3a47f0b5eb2e5d993cc15 "Properly remove useless final states.".

8 years agoFixed 'uniq' on commands (algorithm that removes duplicates).
Ulya Trofimovich [Sat, 3 Dec 2016 00:07:17 +0000 (00:07 +0000)]
Fixed 'uniq' on commands (algorithm that removes duplicates).

The previous version erroneously skipped a coomand after removal,
so it would fail in case of three identical commands in a row.