Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix bugzilla Issue 24861 - Vestige of 16 bit code being accidentally … #17065

Merged
merged 1 commit into from
Nov 15, 2024

Conversation

WalterBright
Copy link
Member

…generated

@WalterBright WalterBright added Bug Fix Easy Review Backend glue code, optimizer, code generation labels Nov 15, 2024
@dlang-bot
Copy link
Contributor

Thanks for your pull request, @WalterBright!

Bugzilla references

Auto-close Bugzilla Severity Description
24861 minor Vestige of 16 bit code being accidentally generated

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#17065"

@braddr
Copy link
Member

braddr commented Nov 15, 2024

Given that the dmc front end is dead, there's no testing of the 16 bit backend any more, right? What's the value of keeping untested/able code? I submitted a pull request a while ago to move all the 16 big code generation, it's quite a sizeable amount of code (if I remember right). Collapsing all those conditionals away would certainly help simplify the remaining code, making it that much easier to understand.

@dlang-bot dlang-bot merged commit a555e13 into dlang:master Nov 15, 2024
41 checks passed
@WalterBright
Copy link
Member Author

@braddr Nice to see you here! I've been working on the Aarch64 backend, and indeed the I16 conditionals can get in the way. Fortunately, it's mostly behind if (I16) and those blocks are easily ignored. In fact, what I'd do for starters is to copy the code gen routines, and remove all the I16, I32, 8087, and XMM code. This made converting what remained to Aarch64 much easier.

@WalterBright WalterBright deleted the fix24861 branch November 15, 2024 03:42
@braddr
Copy link
Member

braddr commented Nov 15, 2024

a little bit rotten:, but not a lot: #16557

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge Backend glue code, optimizer, code generation Bug Fix Easy Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants