Skip to content

Forwardporting "Replace AST-Access by Symbol-Access for hasCardinality()" (#84)#85

Merged
mathias-pfeiffer merged 1 commit intoMontiCore:masterfrom
justusrm:jr/port-cardinality-flag-port-master
Apr 1, 2026
Merged

Forwardporting "Replace AST-Access by Symbol-Access for hasCardinality()" (#84)#85
mathias-pfeiffer merged 1 commit intoMontiCore:masterfrom
justusrm:jr/port-cardinality-flag-port-master

Conversation

@justusrm
Copy link
Copy Markdown
Collaborator

@justusrm justusrm commented Mar 31, 2026

Changed

  • When resolving for adapted variables, ports may have a cardinality and thus be of array type. Previously, the check for cardinalities relied on the AST, which breaks if the symbol is loaded from a serialized symbol table. This change replaces the access to the AST with a newly introduced field on the symbol (and also adds the symbol table completer). This is a forward port from 7.8.30 Extending PortUsageSymbol by hasCardinality flag #84

…iCore#84)

* #190 Extending PortUsageSymbol by hasCardinality flag, using flag in var-resolving
@justusrm justusrm marked this pull request as ready for review March 31, 2026 19:07
@mathias-pfeiffer mathias-pfeiffer changed the title Forwardporting Extending PortUsageSymbol by hasCardinality flag (#84) Forwardporting "Replace AST-Access in Scope Symbol-Access" (#84) Apr 1, 2026
@mathias-pfeiffer mathias-pfeiffer changed the title Forwardporting "Replace AST-Access in Scope Symbol-Access" (#84) Forwardporting "Replace AST-Access by Symbol-Access for hasCardinality()" (#84) Apr 1, 2026
@mathias-pfeiffer mathias-pfeiffer merged commit 1da5dec into MontiCore:master Apr 1, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants