Skip to content

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

Merged

Conversation

nicolasstucki
Copy link
Contributor

Based on #5216

@nicolasstucki nicolasstucki force-pushed the move-inlining-in-reify-quotes branch 2 times, most recently from 22575d7 to cdeaa8d Compare October 11, 2018 08:28
@nicolasstucki nicolasstucki changed the title Move inlining into reify quotes Move inlining into ReifyQuotes and remove Inlined from TASTy Oct 11, 2018
@nicolasstucki nicolasstucki self-assigned this Oct 11, 2018
@nicolasstucki nicolasstucki force-pushed the move-inlining-in-reify-quotes branch from 50a204c to eaf395a Compare October 11, 2018 09:21
@nicolasstucki nicolasstucki force-pushed the move-inlining-in-reify-quotes branch from eaf395a to 6338e53 Compare October 22, 2018 15:39
@nicolasstucki nicolasstucki force-pushed the move-inlining-in-reify-quotes branch 2 times, most recently from be873c9 to 23b1530 Compare October 23, 2018 09:39
@nicolasstucki nicolasstucki force-pushed the move-inlining-in-reify-quotes branch from 23b1530 to cf730b5 Compare October 23, 2018 16:46
Split some neg tests as now only the first error is emitted.
@nicolasstucki nicolasstucki force-pushed the move-inlining-in-reify-quotes branch from cf730b5 to 976e095 Compare October 23, 2018 16:56
@nicolasstucki nicolasstucki requested a review from odersky October 23, 2018 20:12
@allanrenucci allanrenucci mentioned this pull request Oct 24, 2018
8 tasks
Copy link
Contributor

@odersky odersky left a 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
Copy link
Contributor

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe Staging

Copy link
Contributor

@odersky odersky left a 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.

@nicolasstucki nicolasstucki merged commit af4b141 into scala:master Oct 29, 2018
@nicolasstucki nicolasstucki deleted the move-inlining-in-reify-quotes branch October 29, 2018 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants