Skip to content
This repository has been archived by the owner on May 18, 2019. It is now read-only.

[NF] Performance improvements. #2436

Closed
wants to merge 1 commit into from
Closed

[NF] Performance improvements. #2436

wants to merge 1 commit into from

Conversation

perost
Copy link
Member

@perost perost commented May 16, 2018

  • Guard some Error.assertions that otherwise would do lots of
    unnecessary work.
  • Save evaluated bindings.
  • Improve evaluation of subscripted ranges.

@perost
[NF] Performance improvements.
bef6265 
- Guard some Error.assertions that otherwise would do lots of
  unnecessary work.
- Save evaluated bindings.
- Improve evaluation of subscripted ranges.
@OpenModelica-Hudson
Copy link
Member

The test suite is unstable according to OpenModelica_TEST_PULL_REQUEST 2018-05-16_15-01-19.

@OpenModelica-Hudson
Copy link
Member

The tests run correctly according to OpenModelica_TEST_PULL_REQUEST 2018-05-16_15-33-15.

OpenModelica-Hudson pushed a commit that referenced this pull request May 16, 2018
@perost @OpenModelica-Hudson
[NF] Performance improvements.
17adb9a 
- Guard some Error.assertions that otherwise would do lots of
  unnecessary work.
- Save evaluated bindings.
- Improve evaluation of subscripted ranges.

Belonging to [master]:
  - #2436
  - OpenModelica/OpenModelica-testsuite#945
OpenModelica-Hudson pushed a commit to OpenModelica/OpenModelica-testsuite that referenced this pull request May 16, 2018
@perost @OpenModelica-Hudson
[NF] Update tests.
bad53a2 
Belonging to [master]:
  - OpenModelica/OMCompiler#2436
  - #945
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@perost @OpenModelica-Hudson