Conversation
|
Thanks @klickverbot for doing this. |
|
Updated. (No major changes.) |
|
Updated the diff once more (still against 2.067.1). No major changes, although a couple of extra member variables are gone. |
|
How does it look now in D? |
|
@yebblies: Updated (now against v2.070.2). |
src/declaration.d
Outdated
|
|
||
| version(IN_LLVM) | ||
| { | ||
| override void semantic3(Scope *sc) |
There was a problem hiding this comment.
@klickverbot Did this end up having something to do with the llvm linker bug? Or is it still needed?
There was a problem hiding this comment.
@JohanEngelen is the one who did the C++ -> D conversion. However, in this specific case I happen to know that this has been around for ages, and is not related to the linking issue. A year ago or so, I just tried removing the block, and back then, it still made the test suite fail. Maybe the issue has been fixed in the frontend or glue layer since – I don't know, one would have to try.
|
Coincidence that I pushed out my rebase 20 minutes earlier? :-) |
|
OK, so what's the status and outlook of this? |
|
Same as per gdc diff, this is an FYI pr, rather than something that is mergable. I'm not sure if ldc is making any vested interest in reducing the amount of differences between themselves and dmd, due to larger number of people who actively develop there. Which is a shame. |
We do... |
Care to update this then? Or I can go through and say what I think about each individual change. |
|
Updated to 2.077.1 (LDC master) here: #7506 |
|
Cool! I guess we can close this then? |
|
Seems so. |
This pull request is not meant to be merged. It is merely there
for @yebblies and others to judge the current state.