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.

Sign up for GitHub

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

Jump to bottom

Tuple/static array initialization ICE #797

Closed
dnadlinger opened this issue Nov 16, 2014 · 4 comments · Fixed by #798
Closed

Tuple/static array initialization ICE #797

dnadlinger opened this issue Nov 16, 2014 · 4 comments · Fixed by #798
Labels
A-ice

Comments

@dnadlinger
Copy link
Member

Reported by Daniel N on the mailing list:

When compiling the unittest for Peter's graph lib... (on OSX)
https://github.com/Poita/stdex/blob/master/graph.d

I was greeted by the following:
"Stored value type does not match pointer operand type!
  store [2 x i32]* %7, [2 x i32]* %11
 [2 x i32]LLVM ERROR: Broken function found, compilation aborted!"

PR incoming
dnadlinger added a commit to dnadlinger/ldc that referenced this issue Nov 16, 2014
@dnadlinger
Fix initialization of TupleExps with static array members.
d6e54be 
Just as for structs, `DVarValue::getRVal()` doesn't dereference the
address for these.

GitHub: Fixes ldc-developers#797.
@dnadlinger dnadlinger mentioned this issue Nov 16, 2014
Merged
@redstar
Copy link
Member

redstar commented Dec 12, 2014

Fixed in 0.15.1.

@redstar redstar closed this as completed Dec 12, 2014
@dnadlinger
Copy link
Member Author

The PR has not been merged yet, I forgot that it still lacking a test case. Has this been fixed in another way?

@redstar
Copy link
Member

redstar commented Dec 13, 2014

I had the wrong impression that the PR was already merged.

@redstar redstar reopened this Dec 13, 2014
@dnadlinger
Copy link
Member Author

Okay. We can just merge it at any time even without a targeted test case, though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-ice
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix initialization of TupleExps with static array members. dnadlinger/ldc
2 participants
@dnadlinger @redstar