Remove unused RTLSYM.NEWARRAYMTX#21011
Conversation
|
Thanks for your pull request and interest in making D better, @nybzmr! 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#21011" |
ibuclaw
left a comment
There was a problem hiding this comment.
This isn't a bug fix, so please don't target stable. The change needs rebasing as well to remove unrelated commits.
|
@teodutu Could you review this PR when you have time? Let me know if I missed anything. Thanks! |
c8e7cec to
9cb4f68
Compare
|
I have rebased to the latest master. Let me know if you need any further changes :) |
|
Thanks @nybzmr. You can also remove dmd/compiler/src/dmd/backend/rtlsym.d Line 19 in a4cbc08 |
RTLSYM.NEWARRAYMTX was removed as it is no longer used in e2ir.d or anywhere else.