Fix calling convention for functions with D linkage

[ljmf00]

Signed-off-by: Luís Ferreira contact@lsferreira.net

---

Related to dlang/dlang.org#3120 .
CC @kinke .

[dlang-bot]

Thanks for your pull request, @ljmf00!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

Testing this PR locally

If you don't have a local development environment setup, you can use Digger to test this PR:

dub run digger -- build "master + dmd#13287"

[ljmf00]

Some code, beyond my knowledge, is relying on reverse order. The added test runs successfully. I don't know what is happening.

[kinke]

See #5232 for context regarding __xopCmp and its dependence on low-level parameter order.

[ljmf00]

See #5232 for context regarding __xopCmp and its dependence on low-level parameter order.

Thanks, I will test with __xopCmp generated code, although, #7561 got merged, which seems to supersede it. This is kinda messy because I can't have a small test case to reproduce it if it is not related to __xopCmp. If someone can give me a hint I would appreciate it.

[ljmf00]

@MoonlightSentinel I guess C++ interop tests are linking with system druntime/libphobos instead of freshly compiled one. I can't reproduce this. Looking into the logs, DFLAGS are not setting -L-rpath=. forget it, I was using a wrong version of druntime 🙃