-
-
Notifications
You must be signed in to change notification settings - Fork 31.4k
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
gh-109287: Desugar inst(X) to op(X); macro(X) = X #109294
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
f7ea44e
to
ae5b99d
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This doesn't quite work; and there's lots of debug code. But it compiles and passes most tests.
e7f494f
to
f4e954e
Compare
I think super-instructions now work again (the old special-casing was salvageable). I have a few other things:
|
This fixes the crashes with -Xuops I observed.
A few misc cleanups (remove commented-out code etc.)
I'd add @markshannon but he said he's on vacation until end of next week. |
CC @Fidget-Spinner (because I had to change the abstract interpretation code generator a bit). |
|
|
I'm happy with this. The code generator needs to be cleaned up a bit more (e.g. the stats about instrs read/written are weird now).