Project

General

Profile

Actions
Copy link

Bug #5783

closed

Problems with multiple concatenation

Added by Mikhail Chupilko over 9 years ago. Updated almost 5 years ago.

Status:
Closed
Priority:
High
Assignee:
Alexander Kamkin
Category:
nML Translator
Target version:
2.5
Start date:
03/30/2015
Due date:
% Done:

0%

Estimated time:
Detected in build:
2.1.5-beta-150305
Platform:
Published in build:
2.5.1-beta-200127

Description

I'd like to concatenate the signal src<63> 63 times to make a single 64-bit field "top". The response time of this code processing is too high... More than 2-3 minutes. The example follows.

top = src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::
      src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::
      src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::
      src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::
      src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::
      src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::
      src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::
      src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::src<63>::src<63>;

Actions
Copy link
 #1

Updated by Alexander Kamkin over 9 years ago

  • Priority changed from Normal to High
Actions
Copy link
 #2

Updated by Alexander Kamkin over 9 years ago

  • Target version set to 2.2
Actions
Copy link
 #3

Updated by Andrei Tatarnikov over 9 years ago

Can be reproduced even with 24-times concatenation. Parsing tree gets too big (number of possible states increasing) and this significantly slows down parsing. The issue is related to ANTLR and parsing grammar. Need to think how to optimize it.

Actions
Copy link
 #4

Updated by Alexander Kamkin almost 5 years ago

That's funny :).

locationExpr
    :  locationVal (DOUBLE_COLON^ locationExpr)*
    ;

replaced with

locationExpr
    :  locationVal (DOUBLE_COLON^ locationExpr)?
    ;
Actions
Copy link
 #5

Updated by Alexander Kamkin almost 5 years ago

  • Status changed from New to Resolved

That's funny :). Bug is fixed in a couple of minutes.

locationExpr
    :  locationVal (DOUBLE_COLON^ locationExpr)*
    ;

replaced with

locationExpr
    :  locationVal (DOUBLE_COLON^ locationExpr)?
    ;
Actions
Copy link
 #6

Updated by Alexander Kamkin almost 5 years ago

  • Assignee changed from Andrei Tatarnikov to Alexander Kamkin
  • Target version changed from 2.2 to 2.5
Actions
Copy link
 #7

Updated by Alexander Kamkin almost 5 years ago

  • Category set to nML Translator
  • Status changed from Resolved to Closed
  • Published in build set to 2.5.1-beta-200127
Actions
Copy link

Also available in: Atom PDF