-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Move inlining into ReifyQuotes and remove Inlined from TASTy #5241
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
Move inlining into ReifyQuotes and remove Inlined from TASTy #5241
Conversation
22575d7
to
cdeaa8d
Compare
50a204c
to
eaf395a
Compare
eaf395a
to
6338e53
Compare
be873c9
to
23b1530
Compare
23b1530
to
cf730b5
Compare
Split some neg tests as now only the first error is emitted.
cf730b5
to
976e095
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Otherwise LGTM
@@ -2,7 +2,7 @@ package dotty.tools.dotc.decompiler | |||
|
|||
import dotty.tools.dotc.fromtasty._ | |||
import dotty.tools.dotc.core.Phases.Phase | |||
import dotty.tools.dotc.transform.InlineCalls | |||
import dotty.tools.dotc.transform.ReifyQuotes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should think about a name for both inlining and quoting and splicing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe Staging
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Staging is a good name for this.
Based on #5216