New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Platform: Replace non throwing versions of new operator. #5165

Merged
merged 1 commit into from Sep 26, 2017

Conversation

Projects
None yet
5 participants
@pan-
Member

pan- commented Sep 21, 2017

Description

This change prevents inclusion of non throwing versions of the new operator from
the compiler standard library. On GCC, the non throwing version bring with it
some portion of the exception support code.

Inclusion of exception support inflate the RAM required by the standard runtime.

Status

READY

Migrations

NO

Platform: Replace non throwing versions of new operator.
This change prevents inclusion of non throwing versions of the new operator from
the compiler standard library. On GCC, the non throwing version bring with it
some portion of the exception support code.

@pan- pan- requested review from c1728p9 and 0xc0170 Sep 21, 2017

@pan- pan- referenced this pull request Sep 21, 2017

Merged

BLE: Cordio port #5060

@0xc0170 0xc0170 added the needs: CI label Sep 22, 2017

@pan-

This comment has been minimized.

Member

pan- commented Sep 22, 2017

continuous-integration/jenkins/pr-head — Waiting for status to be reported

Is it started or stuck ?

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Sep 22, 2017

@tommikas Can you look at jenkins CI?

@tommikas

This comment has been minimized.

Contributor

tommikas commented Sep 22, 2017

Looks like Jenkins wasn't aware of the PR. I started the repository scan manually and Jenkins found the PR and created the job. It's running now.

@pan-

This comment has been minimized.

Member

pan- commented Sep 25, 2017

@0xc0170 What's missing, how I can help to get this PR in ? It blocks #5060 .

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Sep 25, 2017

@0xc0170 What's missing, how I can help to get this PR in ? It blocks #5060 .

There've been 5.6.1 CI. Lets schedule this one now in the queue

/morph test

@mbed-bot

This comment has been minimized.

mbed-bot commented Sep 25, 2017

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1383

All builds and test passed!

@0xc0170 0xc0170 merged commit 6a46b9c into ARMmbed:master Sep 26, 2017

4 checks passed

Cam-CI uvisor Build & Test Success
Details
ci/morph-test Job has completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@pan- pan- deleted the pan-:non-throwing-new-operator branch Nov 14, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment