Stop emitting fields for inlined fields.

nicolasstucki · nicolasstucki · commit 4d7b2479e85e · 2017-01-10T14:47:47.000+01:00
diff --git a/compiler/src/dotty/tools/dotc/transform/Memoize.scala b/compiler/src/dotty/tools/dotc/transform/Memoize.scala
@@ -95,6 +95,8 @@ import Decorators._
     def adaptToField(tree: Tree) =
       if (tree.isEmpty) tree else tree.ensureConforms(field.info.widen)
 
+    val NoFieldNeeded = Lazy | Deferred | JavaDefined | (if (ctx.settings.YnoInline.value) EmptyFlags else Inline)
+
     if (sym.is(Accessor, butNot = NoFieldNeeded))
       if (sym.isGetter) {
         def skipBlocks(t: Tree): Tree = t match {
@@ -127,5 +129,4 @@ import Decorators._
                 // neither getters nor setters
     else tree
   }
-  private val NoFieldNeeded  = Lazy | Deferred | JavaDefined
 }

Original file line number	Diff line number	Diff line change
`@@ -95,6 +95,8 @@ import Decorators._`
`95`	`95`	`def adaptToField(tree: Tree) =`
`96`	`96`	`if (tree.isEmpty) tree else tree.ensureConforms(field.info.widen)`
`97`	`97`
	`98`	`+ val NoFieldNeeded = Lazy \| Deferred \| JavaDefined \| (if (ctx.settings.YnoInline.value) EmptyFlags else Inline)`
	`99`	`+`
`98`	`100`	`if (sym.is(Accessor, butNot = NoFieldNeeded))`
`99`	`101`	`if (sym.isGetter) {`
`100`	`102`	`def skipBlocks(t: Tree): Tree = t match {`
`@@ -127,5 +129,4 @@ import Decorators._`
`127`	`129`	`// neither getters nor setters`
`128`	`130`	`else tree`
`129`	`131`	`}`
`130`		`- private val NoFieldNeeded = Lazy \| Deferred \| JavaDefined`
`131`	`132`	`}`