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] Backport semanticRun refactoring and recursive semantic SEGV bug fixes #10046

Merged
merged 15 commits into from
Jun 16, 2019

Conversation

ibuclaw
Copy link
Member

@ibuclaw ibuclaw commented Jun 16, 2019

They are all semi-interdependent on each other leading up to the segfault that I'm wanting to squash.

@dlang-bot
Copy link
Contributor

dlang-bot commented Jun 16, 2019

Thanks for your pull request, @ibuclaw!

Bugzilla references

Auto-close Bugzilla Severity Description
15875 normal case of undetected circular reference in function parameter
16214 regression [REG2.069] ICE: Assertion `fd->semanticRun == PASSsemantic3done' failed.
17285 normal Segfault when types are used in array literals
17290 normal DMD crash due to circular reference in function return type
17686 regression [REG2.075.0] Covariant return types doesn't work with override in some cases
19336 normal [ICE] segfault on invalid code
19691 regression ICE on null default value for struct parameter in constructor
19717 normal case of segfault due to undetected forward reference

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

@ibuclaw ibuclaw changed the title [dmd-cxx] semanticRun refactoring and recursive semantic SEGV bug fixes [dmd-cxx] Backport semanticRun refactoring and recursive semantic SEGV bug fixes Jun 16, 2019
@ibuclaw
Copy link
Member Author

ibuclaw commented Jun 16, 2019

There's a couple apparent regressions, just going to look into them...

@ibuclaw
Copy link
Member Author

ibuclaw commented Jun 16, 2019

There was only one regression, the other was a test missing EXTRA_SOURCES option, probably needs fixing in master too for when I stage the dmd-master branch in gcc.

@ibuclaw ibuclaw merged commit 792f0fd into dlang:dmd-cxx Jun 16, 2019
@ibuclaw ibuclaw deleted the dmd-cxx-semantic branch June 16, 2019 20:57
kraj pushed a commit to kraj/gcc that referenced this pull request Jun 16, 2019
d/dmd: Merge upstream dmd 792f0fdf2

Fixes segmentation fault in functionParameters, and other related
semantic bugs in forward or recursively referenced declarations.

Reviewed-on: dlang/dmd#10046

git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@272366 138bc75d-0d04-0410-961f-82ee72b054a4
asiekierka pushed a commit to WonderfulToolchain/gcc-ia16 that referenced this pull request May 16, 2022
	PR d/90603
d/dmd: Merge upstream dmd 792f0fdf2

Fixes segmentation fault in functionParameters, and other related
semantic bugs in forward or recursively referenced declarations.

Reviewed-on: dlang/dmd#10046

From-SVN: r272366
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.

6 participants