Skip to content
  • Categories
  • Recent
  • Tags
  • Popular
  • World
  • Users
  • Groups
Skins
  • Light
  • Cerulean
  • Cosmo
  • Flatly
  • Journal
  • Litera
  • Lumen
  • Lux
  • Materia
  • Minty
  • Morph
  • Pulse
  • Sandstone
  • Simplex
  • Sketchy
  • Spacelab
  • United
  • Yeti
  • Zephyr
  • Dark
  • Cyborg
  • Darkly
  • Quartz
  • Slate
  • Solar
  • Superhero
  • Vapor

  • Default (No Skin)
  • No Skin
Collapse
Code Project
  1. Home
  2. The Lounge
  3. I might need to optimize this XD

I might need to optimize this XD

Scheduled Pinned Locked Moved The Lounge
regexquestioncode-review
24 Posts 6 Posters 0 Views 1 Watching
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • H honey the codewitch

    I'm not using any foreach loops. I've already optimized the VM itself to within an inch of its life

    Real programmers use butterflies

    J Offline
    J Offline
    Jorgen Andersson
    wrote on last edited by
    #14

    How about Linq?

    Wrong is evil and must be defeated. - Jeff Ello

    H 1 Reply Last reply
    0
    • J Jorgen Andersson

      How about Linq?

      Wrong is evil and must be defeated. - Jeff Ello

      H Offline
      H Offline
      honey the codewitch
      wrote on last edited by
      #15

      I thought the goal was to speed this up?

      Real programmers use butterflies

      J 1 Reply Last reply
      0
      • H honey the codewitch

        L0001: jmp L0002, L0010, L0021, L0029, L0040, L0050, L0059, L0070, L0082, L0092, L0101, L0109, L0115, L0121, L0133, L0142, L0149, L0157, L0167, L0174, L0185, L0190, L0198, L0209, L0213, L0219, L0223, L0233, L0241, L0247, L0256, L0264, L0272, L0278, L0285, L0292, L0297, L0302, L0306, L0311, L0315, L0319, L0323, L0327, L0331, L0335, L0339, L0343, L0347, L0352, L0357, L0361, L0366, L0371, L0375, L0380, L0384, L0389, L0393, L0398, L0402, L0407, L0412, L0416, L0421, L0426, L0430, L0435, L0440, L0444, L0449, L0453, L0458, L0492, L0499, L0506, L0513, L0520, L0527, L0537, L0546, L0554, L0561, L0568, L0574, L0583, L0591, L0598, L0606, L0616, L0625, L0633, L0640, L0647, L0656, L0665, L0671, L0681, L0690, L0696, L0702, L0708, L0716, L0728, L0753, L0768, L0773

        Each JMP operand spawns a fiber (basically a thread). I haven't counted how many are spawned here, but 70-80 or so? I think maybe this code is a bit heavy handed. This is just to match a (quite complicated) regular expression

        Real programmers use butterflies

        Z Offline
        Z Offline
        ZTransform
        wrote on last edited by
        #16

        Lounge?

        H J 2 Replies Last reply
        0
        • H honey the codewitch

          I thought the goal was to speed this up?

          Real programmers use butterflies

          J Offline
          J Offline
          Jorgen Andersson
          wrote on last edited by
          #17

          I didn't tell you to use it, I'm just looking for problems. :-)

          Wrong is evil and must be defeated. - Jeff Ello

          H 1 Reply Last reply
          0
          • Z ZTransform

            Lounge?

            H Offline
            H Offline
            honey the codewitch
            wrote on last edited by
            #18

            Yes, this is the Lounge.

            Real programmers use butterflies

            1 Reply Last reply
            0
            • J Jorgen Andersson

              I didn't tell you to use it, I'm just looking for problems. :-)

              Wrong is evil and must be defeated. - Jeff Ello

              H Offline
              H Offline
              honey the codewitch
              wrote on last edited by
              #19

              Look away. Here's almost all of it. The stuff you don't see is very thin

              public static int Run(int[][] prog,LexContext input)
              {
              input.EnsureStarted();
              int i,match=-1;
              _Fiber[] currentFibers, nextFibers, tmp;
              int currentFiberCount=0, nextFiberCount=0;
              int[] pc;
              // position in input
              int sp=0;
              // stores our captured input
              var sb = new StringBuilder(64);
              int[] saved, matched;
              saved = new int[2];
              currentFibers = new _Fiber[prog.Length];
              nextFibers = new _Fiber[prog.Length];
              _EnqueueFiber(ref currentFiberCount, ref currentFibers, new _Fiber(prog,0, saved), 0);
              matched = null;
              var cur = -1;
              if (LexContext.EndOfInput != input.Current)
              {
              var ch1 = unchecked((char)input.Current);
              if (char.IsHighSurrogate(ch1))
              {
              if (-1 == input.Advance())
              throw new ExpectingException("Expecting low surrogate in unicode stream. The input source is corrupt or not valid Unicode", input.Line, input.Column, input.Position, input.FileOrUrl);
              var ch2 = unchecked((char)input.Current);
              cur = char.ConvertToUtf32(ch1, ch2);
              }
              else
              cur = ch1;

              }
              		
              while(0
              
              J 1 Reply Last reply
              0
              • Z ZTransform

                Lounge?

                J Offline
                J Offline
                Jorgen Andersson
                wrote on last edited by
                #20

                In the sticky post at the top: 2. Technical discussions are welcome...[^]

                Wrong is evil and must be defeated. - Jeff Ello

                1 Reply Last reply
                0
                • H honey the codewitch

                  Look away. Here's almost all of it. The stuff you don't see is very thin

                  public static int Run(int[][] prog,LexContext input)
                  {
                  input.EnsureStarted();
                  int i,match=-1;
                  _Fiber[] currentFibers, nextFibers, tmp;
                  int currentFiberCount=0, nextFiberCount=0;
                  int[] pc;
                  // position in input
                  int sp=0;
                  // stores our captured input
                  var sb = new StringBuilder(64);
                  int[] saved, matched;
                  saved = new int[2];
                  currentFibers = new _Fiber[prog.Length];
                  nextFibers = new _Fiber[prog.Length];
                  _EnqueueFiber(ref currentFiberCount, ref currentFibers, new _Fiber(prog,0, saved), 0);
                  matched = null;
                  var cur = -1;
                  if (LexContext.EndOfInput != input.Current)
                  {
                  var ch1 = unchecked((char)input.Current);
                  if (char.IsHighSurrogate(ch1))
                  {
                  if (-1 == input.Advance())
                  throw new ExpectingException("Expecting low surrogate in unicode stream. The input source is corrupt or not valid Unicode", input.Line, input.Column, input.Position, input.FileOrUrl);
                  var ch2 = unchecked((char)input.Current);
                  cur = char.ConvertToUtf32(ch1, ch2);
                  }
                  else
                  cur = ch1;

                  }
                  		
                  while(0
                  
                  J Offline
                  J Offline
                  Jorgen Andersson
                  wrote on last edited by
                  #21

                  Oh, no wonder then, you're doing it on purpose... :laugh:

                  Wrong is evil and must be defeated. - Jeff Ello

                  H 1 Reply Last reply
                  0
                  • J Jorgen Andersson

                    Oh, no wonder then, you're doing it on purpose... :laugh:

                    Wrong is evil and must be defeated. - Jeff Ello

                    H Offline
                    H Offline
                    honey the codewitch
                    wrote on last edited by
                    #22

                    Doing what on purpose? I'm a little slow this morning. :)

                    Real programmers use butterflies

                    J 1 Reply Last reply
                    0
                    • H honey the codewitch

                      Doing what on purpose? I'm a little slow this morning. :)

                      Real programmers use butterflies

                      J Offline
                      J Offline
                      Jorgen Andersson
                      wrote on last edited by
                      #23

                      Spawning loads of fibers. Or is this auto generated code again?

                      Wrong is evil and must be defeated. - Jeff Ello

                      H 1 Reply Last reply
                      0
                      • J Jorgen Andersson

                        Spawning loads of fibers. Or is this auto generated code again?

                        Wrong is evil and must be defeated. - Jeff Ello

                        H Offline
                        H Offline
                        honey the codewitch
                        wrote on last edited by
                        #24

                        Well, it's not on purpose per se. I mean yes, I'm spawning a lot of them, but the idea is to keep as few active or "alive" at one time as possible. when I see a jmp with 3 operands it spawns 2 fibers in addition to a primary fiber. That's what I don't want, since every fiber has to examine the character under the cursor which leads to many examinations of the same character. There's no way to optimize this out because it's rather the point of the fiber running in the first place. Multiple examinations are a byproduct of the NFA algorithm. My goal is simply to reduce/eliminate the amount of jmps and especially the number of operands they have. A pure DFA can run by examining each character only once.

                        Real programmers use butterflies

                        1 Reply Last reply
                        0
                        Reply
                        • Reply as topic
                        Log in to reply
                        • Oldest to Newest
                        • Newest to Oldest
                        • Most Votes


                        • Login

                        • Don't have an account? Register

                        • Login or register to search.
                        • First post
                          Last post
                        0
                        • Categories
                        • Recent
                        • Tags
                        • Popular
                        • World
                        • Users
                        • Groups