diff options
author | Adriaan Moors <adriaan.moors@typesafe.com> | 2013-06-26 11:15:08 -0700 |
---|---|---|
committer | Adriaan Moors <adriaan.moors@typesafe.com> | 2013-06-26 11:15:08 -0700 |
commit | deb88538116ab0f1fe598c91e4d026a83ecc5404 (patch) | |
tree | 61253772d79c93c7aa7142e325ea3e0654d162b1 /test | |
parent | d22e8d282c538a711f6fdd33793c06cd7676959d (diff) | |
parent | 228549356ce5dda0305dc2e9e08870859e4de06e (diff) | |
download | scala-deb88538116ab0f1fe598c91e4d026a83ecc5404.tar.gz scala-deb88538116ab0f1fe598c91e4d026a83ecc5404.tar.bz2 scala-deb88538116ab0f1fe598c91e4d026a83ecc5404.zip |
Merge pull request #2655 from VladUreche/issue/7344
SI-7344 Specialize methods in private scopes
Diffstat (limited to 'test')
-rw-r--r-- | test/files/specialized/SI-7344.scala | 53 |
1 files changed, 53 insertions, 0 deletions
diff --git a/test/files/specialized/SI-7344.scala b/test/files/specialized/SI-7344.scala new file mode 100644 index 0000000000..1040460bd1 --- /dev/null +++ b/test/files/specialized/SI-7344.scala @@ -0,0 +1,53 @@ +/* Test for SI-7344, where specialized methods inside the bodies of other + * methods are not specialized, although they might as well be. The name + * for the specialized method should not be different depending on the + * outside method/class' specialization. */ + +class Test[@specialized(Int, Double) X](val x: X) { + + def checkSpecialization[Y](@specialized(Int, Double) y: Y): X = { + + // checking the specialization using the method name, which we can + // extract from an exception's stack trace. We can match just the + // prefix, since the compiler will add a suffix to the method name + // during lambdalift, when it lifts the local methods outside. + def specMe[@specialized(Int, Double) T, N](t: T, n: N): Unit = checkNameStartsWith(n.toString) + + // expected to specialize: + specMe("x", "specMe") + specMe(123, "specMe$mIc$sp") + specMe(1.3, new { override def toString = "specMe$mDc$sp" }) + + x + } + + // name matching: + private[this] def checkNameStartsWith(prefix: String): Unit = { + val method = (new Exception).getStackTrace()(1).getMethodName() + assert(method.startsWith(prefix), method + ".startsWith(" + prefix + ") should be true") + } +} + +object Test extends App { + val t1 = new Test("x") + val t2 = new Test(123) + val t3 = new Test(1.3) + + // we want specialization to rewire these, + // that's why they're not in a for loop: + t1.checkSpecialization("x") + + // Prevented by SI-7579: + // The duplicator loses the @specialized annotation, + // so our tree transformation doesn't know it needs to + // specialize specMe inside the duplicated (and specialized) + // variants of the `checkSpecialization` method + // t1.checkSpecialization(123) + // t1.checkSpecialization(1.3) + // t2.checkSpecialization("x") + // t2.checkSpecialization(123) + // t2.checkSpecialization(1.3) + // t3.checkSpecialization("x") + // t3.checkSpecialization(123) + // t3.checkSpecialization(1.3) +} |