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

Issue 14459 - String literal merge bug causes incorrect runtime program behavior #4866

Merged
merged 2 commits into from Aug 6, 2015

Conversation

9rnsr
Copy link
Contributor

@9rnsr 9rnsr commented Aug 6, 2015

https://issues.dlang.org/show_bug.cgi?id=14459

The excessive value expansion in optimize.c had corrupted the result of pointer identity comparison.

… is e2`

By this additional translation, we can have only one semantic analysis result for the equivalent expression forms.
@9rnsr
Copy link
Contributor Author

9rnsr commented Aug 6, 2015

Also, I found a small AST ambiguity between EqualExp and IdentityExp for pointer comparisons.
It's not a bug, but the additional translation and unification will reduce maintenance cost.

@dnadlinger
Copy link
Member

Auto-merge toggled on

dnadlinger added a commit that referenced this pull request Aug 6, 2015
Issue 14459 - String literal merge bug causes incorrect runtime program behavior
@dnadlinger dnadlinger merged commit fd8e527 into dlang:master Aug 6, 2015
@9rnsr 9rnsr deleted the fix14459 branch August 7, 2015 01:11
9rnsr pushed a commit to 9rnsr/dmd that referenced this pull request Aug 7, 2015
Issue 14459 - String literal merge bug causes incorrect runtime program behavior
9rnsr pushed a commit to 9rnsr/dmd that referenced this pull request Aug 10, 2015
Issue 14459 - String literal merge bug causes incorrect runtime program behavior
@MoonlightSentinel
Copy link
Contributor

This PR caused a regression: https://issues.dlang.org/show_bug.cgi?id=20758

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants