Skip to content

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

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

Merged
merged 1 commit into from
Sep 26, 2017

Conversation

pan-
Copy link
Member

@pan- 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

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 September 21, 2017 17:43
@pan- pan- mentioned this pull request Sep 21, 2017
@pan-
Copy link
Member Author

pan- commented Sep 22, 2017

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

Is it started or stuck ?

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 22, 2017

@tommikas Can you look at jenkins CI?

@tommikas
Copy link
Contributor

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-
Copy link
Member Author

pan- commented Sep 25, 2017

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

@0xc0170
Copy link
Contributor

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
Copy link

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
@pan- pan- deleted the non-throwing-new-operator branch November 14, 2018 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants