diff options
author | Jason Zaugg <jzaugg@gmail.com> | 2013-08-18 12:32:17 +0200 |
---|---|---|
committer | Jason Zaugg <jzaugg@gmail.com> | 2013-08-29 10:44:51 +0200 |
commit | 26dfa54ef31687e4c3e5b85be476ee5759f77950 (patch) | |
tree | 75779aa9cf7a4d97e8e06f69f8cc7418e5963040 /test/files/run/t3508.scala | |
parent | a3fad0d1d42b4af4b506ce167209bdeea5cd9d5c (diff) | |
download | scala-26dfa54ef31687e4c3e5b85be476ee5759f77950.tar.gz scala-26dfa54ef31687e4c3e5b85be476ee5759f77950.tar.bz2 scala-26dfa54ef31687e4c3e5b85be476ee5759f77950.zip |
SI-7763 Avoid dropping casts in erasure
466b7d29f avoided quadratic complexity in Erasure's treatment
of chained `asInstanceOf` calls. It did so by using the typechecked
qualifier, rather than discarding it.
However, that also dropped the cast altogether! In many cases this
was masked by later inclusion of a cast to the expected type
by `adaptToType`:
at scala.tools.nsc.transform.Erasure$Eraser.cast(Erasure.scala:636)
at scala.tools.nsc.transform.Erasure$Eraser.scala$tools$nsc$transform$Erasure$Eraser$$adaptToType(Erasure.scala:665)
at scala.tools.nsc.transform.Erasure$Eraser.adapt(Erasure.scala:766)
at scala.tools.nsc.typechecker.Typers$Typer.runTyper$1(Typers.scala:5352)
This commit re-wraps the typechecked `qual` in its original
`<qual>.asInstanceOf[T]` to preserve semantics while avoiding
the big-O blowup.
The test includes the compiler option `-Ynooptimize` because dead code
elimination *also* thinks that this cast is superfluous. I'll follow up
on that problem seprately.
Diffstat (limited to 'test/files/run/t3508.scala')
0 files changed, 0 insertions, 0 deletions