Skip to content
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

[dmd-cxx] Use rmem instead of libc for malloc/strdup #8913

Merged
merged 1 commit into from
Nov 4, 2018

Conversation

ibuclaw
Copy link
Member

@ibuclaw ibuclaw commented Nov 4, 2018

As proper checking + error handling of the return value is managed.

@ibuclaw ibuclaw added GDC Gnu D Compiler Trivial typos, formatting, comments C++ Port labels Nov 4, 2018
@dlang-bot
Copy link
Contributor

Thanks for your pull request, @ibuclaw!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

Testing this PR locally

If you don't have a local development environment setup, you can use Digger to test this PR:

dub fetch digger
dub run digger -- build "dmd-cxx + dmd#8913"

@ibuclaw
Copy link
Member Author

ibuclaw commented Nov 4, 2018

Merging to redirect the last fixes over to #8910.

@ibuclaw ibuclaw merged commit 497a814 into dlang:dmd-cxx Nov 4, 2018
@ibuclaw ibuclaw deleted the dmd-cxx-memmalloc branch November 4, 2018 13:16
kraj pushed a commit to kraj/gcc that referenced this pull request Nov 5, 2018
This introduces a new header that pulls in system includes for use only
in the DMD front-end part of the compiler, fixing up uses of problematic
functions that are prevalent throughout the code.

Commits merged from dmd.

    Fix build of the D frontend on the Hurd and KFreeBSD.
    Initial patch from Matthias Klose.
    dlang/dmd#8893

    Don't care about D/C++ compatibility in C++ port.
    Fixes build error in https://gcc.gnu.org/PR87788
    dlang/dmd#8895

    Allow compiling front-end headers with strict warnings.
    dlang/dmd#8909

    Add root/system.h header for wrapping system includes.
    Fixes https://gcc.gnu.org/PR87865
    dlang/dmd#8910

    Move checkedint to dmd/root.
    dlang/dmd#8912

    Use rmem instead of libc for malloc() and strdup().
    dlang/dmd#8913

    Use align(8) for alignment of UnionExp, fixing several BUS errors
    due to alignment issues on SPARC.
    dlang/dmd#8914

    Don't pass NULL pointer as format parameter to errorSupplemental.
    dlang/dmd#8916

gcc/d/ChangeLog:

2018-11-05  Iain Buclaw  <ibuclaw@gdcproject.org>

	PR d/87865
	* d-system.h: New file.

git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@265780 138bc75d-0d04-0410-961f-82ee72b054a4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C++ Port GDC Gnu D Compiler Trivial typos, formatting, comments
Projects
None yet
3 participants