Improve C++ layout test error messages#23190
Conversation
|
Thanks for your pull request and interest in making D better, @dkorpel! 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 referencesYour 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 locallyIf you don't have a local development environment setup, you can use Digger to test this PR: dub run digger -- build "master + dmd#23190" |
8cf2b85 to
4d7db1f
Compare
Instead of a default static assert message, give the values and locations of the mismatches symbols. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
4d7db1f to
b989ce3
Compare
|
Verified that the CI runs: |
can you demangle those symbols with druntime and |
|
The problem is that it's currently a compile time only test using static assert, and druntime's demangler only implements extern(D) demangling (it hooks to |
if that fails surely it will print on stderr, which you can pipe to |
Instead of a default static assert message, give the values and locations
of the mismatches symbols.
Also important: pass
-unittestin the build step so those checks are actually performed.