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] fix Issue 19734 and 19735 #9452

Merged
merged 2 commits into from Mar 15, 2019
Merged

Conversation

ibuclaw
Copy link
Member

@ibuclaw ibuclaw commented Mar 15, 2019

Backport for ICE.

@ibuclaw ibuclaw added GDC Gnu D Compiler C++ Port labels Mar 15, 2019
@dlang-bot
Copy link
Contributor

Thanks for your pull request, @ibuclaw!

Bugzilla references

Auto-close Bugzilla Severity Description
19734 blocker isDataseg returns true for non-static declarations
19735 critical Error: variable extern symbols cannot have initializers

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#9452"

@ibuclaw
Copy link
Member Author

ibuclaw commented Mar 15, 2019

@braddr - Windows still fails checking out sources.

Bootstrapped and regression tested, so merging...

@ibuclaw ibuclaw merged commit 8d4c876 into dlang:dmd-cxx Mar 15, 2019
@ibuclaw ibuclaw deleted the dmd-cxx-issue19734 branch March 15, 2019 11:52
kraj pushed a commit to kraj/gcc that referenced this pull request Mar 15, 2019
d/dmd: Merge upstream dmd 8d4c876c6

The extern storage class flag was wrongly propagated to function scope
when starting the semantic pass on the body.

Fixes https://gcc.gnu.org/PR88990

Reviewed-on: dlang/dmd#9452

git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@269708 138bc75d-0d04-0410-961f-82ee72b054a4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants