Project

General

Profile

Bug #5783

Problems with multiple concatenation

Added by Mikhail Chupilko over 5 years ago. Updated 6 months 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>;

Associated revisions

Revision 411d0c61 (diff)
Added by Alexander Kamkin 7 months ago

Fixed Bug #5783.

Signed-off-by: Alexander Kamkin <>

History

#1

Updated by Alexander Kamkin over 5 years ago

  • Priority changed from Normal to High
#2

Updated by Alexander Kamkin over 5 years ago

  • Target version set to 2.2
#3

Updated by Andrei Tatarnikov over 5 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.

#4

Updated by Alexander Kamkin 7 months ago

That's funny :).

locationExpr
    :  locationVal (DOUBLE_COLON^ locationExpr)*
    ;

replaced with

locationExpr
    :  locationVal (DOUBLE_COLON^ locationExpr)?
    ;
#5

Updated by Alexander Kamkin 7 months 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)?
    ;
#6

Updated by Alexander Kamkin 7 months ago

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

Updated by Alexander Kamkin 6 months ago

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

Also available in: Atom PDF