]> Savannah Git Hosting - gnulib.git/commitdiff
dfa: optimization for state merge
authorNorihiro Tanaka <noritnk@kcn.ne.jp>
Wed, 19 Sep 2018 15:35:49 +0000 (08:35 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Wed, 19 Sep 2018 15:36:44 +0000 (08:36 -0700)
* lib/dfa.c (merge2): New function.
(merge_nfa_state): Use it.

ChangeLog
lib/dfa.c

index 181039c1a1604187e2d3fdc8feed1e7eb5f8c9fe..b9d7b30d6aca2277b28bedf576ffff74acdc4df3 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2018-09-19  Norihiro Tanaka  <noritnk@kcn.ne.jp>
+
+       dfa: optimization for state merge
+       * lib/dfa.c (merge2): New function.
+       (merge_nfa_state): Use it.
+
 2018-09-18  Jim Meyering  <meyering@fb.com>
 
        dfa: trivial comment fix: s/is/if/
index 13b2a0bafb0d9d790273aa7db31782b77e30ca45..d04bcbe6407bb3506bde62969454b7c5b0bda72f 100644 (file)
--- a/lib/dfa.c
+++ b/lib/dfa.c
@@ -2102,6 +2102,21 @@ merge (position_set const *s1, position_set const *s2, position_set *m)
   merge_constrained (s1, s2, -1, m);
 }
 
+static void
+merge2 (position_set *dst, position_set const *src, position_set *m)
+{
+  if (src->nelem < 4)
+    {
+      for (ptrdiff_t i = 0; i < src->nelem; ++i)
+        insert (src->elems[i], dst);
+    }
+   else
+    {
+      merge (src, dst, m);
+      copy (m, dst);
+    }
+}
+
 /* Delete a position from a set.  Return the nonzero constraint of the
    deleted position, or zero if there was no such position.  */
 static unsigned int
@@ -2388,8 +2403,7 @@ merge_nfa_state (struct dfa *d, size_t *queue, ptrdiff_t nqueue, size_t tindex,
           if (flags[sindex] & OPT_REPEAT)
             delete (sindex, &follows[sindex]);
 
-          merge (&follows[dindex], &follows[sindex], merged);
-          copy (merged, &follows[dindex]);
+          merge2 (&follows[dindex], &follows[sindex], merged);
 
           /* Mark it as pruned in future.  */
           follows[tindex].elems[j].constraint = 0;