Conversation
|
Thanks for your pull request and interest in making D better, @kinke! We are looking forward to reviewing it, and you should be hearing from a maintainer soon.
Please see CONTRIBUTING.md for more information. If you have addressed all reviews or aren't sure how to proceed, don't hesitate to ping us with a simple comment. Bugzilla references
Testing this PR locallyIf you don't have a local development environment setup, you can use Digger to test this PR: dub run digger -- build "master + dmd#13593" |
Leading to a swapped `rhs.opEquals(lhs)` check *if* the opEquals method took its parameter by ref.
|
auto-tester failing is to be expected, as it doesn't support cloning same-named druntime/Phobos branches for PRs originating from the official repo. AFAIK, Azure and Cirrus already did support that; CircleCI newly does with an extra commit here. Buildkite needs something like dlang/ci#452; only LDC should fail. |
druntime has been adapted accordingly too.
…g opEquals order Affecting the frontend itself, see https://issues.dlang.org/show_bug.cgi?id=22717. Preparing LDC early is required to make DMD's Buildkite CI pass with the proposed fix in dlang/dmd#13593.
|
I've changed the fix to only apply if the compiler is going to define |
|
Buildkite still needs to bump its LDC ref once it's prepared for the upcoming breaking change (ldc-developers/ldc#3910), then this PR is ready. |
…g opEquals order (#3910) Affecting the frontend itself, see https://issues.dlang.org/show_bug.cgi?id=22717. Preparing LDC early is required to make DMD's Buildkite CI pass with the proposed fix in dlang/dmd#13593.
…als order change This prepares Buildkite for dlang/dmd#13593.
…als order change This prepares Buildkite for dlang/dmd#13593.
|
Ready now. Note: all compilers with |
|
Well we never claimed that newer versions could build older - and, usually it's deprecations and import bugfixes that mean we can't go back more than a couple anyway without having to downgrade the host compiler. ;-) |
Leading to a swapped
rhs.opEquals(lhs)check if theopEqualsmethod took its parameter by ref.This depends on dlang/druntime#3718.