Skip to content

Conversation

ccli8
Copy link
Contributor

@ccli8 ccli8 commented Dec 10, 2019

Summary of changes

This PR is continuation of #12047 and tries to fix static SRAM OOM further by supporting dynamic heap configuration on IAR. In this PR, for IAR toolchain, heap is configured to 1KiB at a minimum and expandable to available SRAM. This requires IAR 8.x.


Pull request type

[x] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

Test results

[x] No Tests required for this change (E.g docs only update)
[] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

Reviewers

@mprse @0xc0170


On IAR, configure heap to 1KiB at a minimum and expandable, dependent on available SRAM. This requires IAR 8.x.
@ciarmcom
Copy link
Member

@ccli8, thank you for your changes.
@mprse @0xc0170 @Ronny-Liu @ARMmbed/mbed-os-maintainers please review.

@0xc0170 0xc0170 added release-version: 6.0.0-alpha-1 First pre-release version of 6.0.0 needs: CI and removed needs: review labels Dec 10, 2019
@mbed-ci
Copy link

mbed-ci commented Dec 10, 2019

Test run: FAILED

Summary: 4 of 4 test jobs failed
Build number : 1
Build artifacts

Failed test jobs:

  • jenkins-ci/mbed-os-ci_unittests
  • jenkins-ci/mbed-os-ci_build-GCC_ARM
  • jenkins-ci/mbed-os-ci_build-ARM
  • jenkins-ci/mbed-os-ci_build-IAR

@0xc0170
Copy link
Contributor

0xc0170 commented Dec 11, 2019

Will restart the tests (internal fault)

@0xc0170
Copy link
Contributor

0xc0170 commented Dec 11, 2019

CI restarted

@mbed-ci
Copy link

mbed-ci commented Dec 11, 2019

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 2
Build artifacts

@0xc0170 0xc0170 removed the needs: CI label Dec 11, 2019
@0xc0170 0xc0170 merged commit 1d5bd75 into ARMmbed:master Dec 11, 2019
@cyliangtw cyliangtw deleted the nuvoton_nano130_iar_dynamic_heap branch March 9, 2023 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-version: 6.0.0-alpha-1 First pre-release version of 6.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants