Skip to content

Commit

Permalink
Update opt options in test
Browse files Browse the repository at this point in the history
  • Loading branch information
som-snytt committed Nov 22, 2021
1 parent 2624806 commit dc9b785
Show file tree
Hide file tree
Showing 57 changed files with 57 additions and 58 deletions.
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:** -Wopt
package instrumented

object MyPredef {
Expand Down
2 changes: 1 addition & 1 deletion test/files/instrumented/inline-in-constructors/bar_2.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
package instrumented

/** Class that uses assert compiled in previous compiler run so we check if
Expand Down
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:** -Wopt
import scala.tools.partest.instrumented.Instrumentation._
import instrumented._

Expand Down
2 changes: 1 addition & 1 deletion test/files/neg/inlineIndyLambdaPrivate/Test_2.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:** -Yopt-inline-heuristics:everything -opt-warnings:_ -Werror
// scalac: -opt:inline:** -Yopt-inline-heuristics:everything -Wopt -Werror
class Test {
def foo = A_1.test
}
2 changes: 1 addition & 1 deletion test/files/neg/sealed-final-neg.scala
@@ -1,4 +1,4 @@
// scalac: -Xfatal-warnings -opt:l:inline -opt-inline-from:** -opt-warnings
// scalac: -Werror -opt:inline:** -Wopt
//
package neg1 {
sealed abstract class Foo {
Expand Down
2 changes: 1 addition & 1 deletion test/files/neg/t11746.scala
@@ -1,5 +1,5 @@
//
// scalac: -Werror -opt:l:inline -opt-inline-from:'**' -opt-warnings:none,_
// scalac: -Werror -opt:inline:** -Wopt:none
//
// compare -opt-warnings:none,at-inline-failed-summary

Expand Down
2 changes: 1 addition & 1 deletion test/files/pos/inline-access-levels/A_1.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:** -opt-warnings -Werror
// scalac: -opt:inline:** -Wopt -Werror
package test

object A {
Expand Down
2 changes: 1 addition & 1 deletion test/files/pos/inline-access-levels/Test_2.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:** -opt-warnings -Werror
// scalac: -opt:inline:** -Wopt -Werror
package test

object Test {
Expand Down
2 changes: 1 addition & 1 deletion test/files/pos/t11663/B_2.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:** -opt-warnings:_
// scalac: -opt:inline:** -Wopt
class B {
def bar(c: A) = c.m
}
2 changes: 1 addition & 1 deletion test/files/pos/t3234.scala
@@ -1,5 +1,5 @@

// scalac: -opt:l:inline -opt-inline-from:** -opt-warnings -Xfatal-warnings
// scalac: -opt:inline:** -Wopt -Werror
//
trait Trait1 {
@inline final def foo2(n: Int) = n*n
Expand Down
2 changes: 1 addition & 1 deletion test/files/pos/t3420.scala
@@ -1,5 +1,5 @@

// scalac: -opt-warnings -opt:l:inline -opt-inline-from:** -Xfatal-warnings
// scalac: -opt:inline:** -Wopt -Werror
//
class C {
val cv = Map[Int, Int](1 -> 2)
Expand Down
2 changes: 1 addition & 1 deletion test/files/pos/t4840.scala
@@ -1,5 +1,5 @@

// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:** -Wopt
//
class Crashy {
def g(): Option[Any] = None
Expand Down
2 changes: 1 addition & 1 deletion test/files/pos/t8410.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:** -Xfatal-warnings -deprecation:false -opt-warnings:none
// scalac: -opt:inline:** -Wopt:none -Werror -deprecation:false
//

object Test extends App {
Expand Down
2 changes: 1 addition & 1 deletion test/files/pos/t9111-inliner-workaround/Test_1.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:** -Wopt
//
object Test extends App {
println(new A_1.Inner())
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/bcodeInlinerMixed/B_1.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
//
// Since 1.0.18, partest does mixed compilation only in two stages
// 1. scalac *.scala *.java
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/bcodeInlinerMixed/Test_2.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
//
import scala.tools.partest.BytecodeTest
import scala.tools.testkit.ASMConverters
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/classfile-format-51.scala
Expand Up @@ -17,7 +17,7 @@ import Opcodes._
// verify. So the test includes a version check that short-circuits the whole test
// on JDK 6
object Test extends DirectTest {
override def extraSettings: String = s"-opt:l:inline -opt-inline-from:** -usejavacp -cp ${testOutput.path}"
override def extraSettings: String = s"-opt:inline:** -usejavacp -cp ${testOutput.path}"

def generateClass(): Unit = {
val invokerClassName = "DynamicInvoker"
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/classfile-format-52.scala
Expand Up @@ -14,7 +14,7 @@ import Opcodes._
// By its nature the test can only work on JDK 8+ because under JDK 7- the
// interface won't verify.
object Test extends DirectTest {
override def extraSettings: String = s"-opt:l:inline -opt-inline-from:** -usejavacp -cp ${testOutput.path}"
override def extraSettings: String = s"-opt:inline:** -usejavacp -cp ${testOutput.path}"

def generateInterface(): Unit = {
val interfaceName = "HasDefaultMethod"
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/icode-reader-dead-code.scala
Expand Up @@ -36,7 +36,7 @@ object Test extends DirectTest {

// If inlining fails, the compiler will issue an inliner warning that is not present in the
// check file
compileString(newCompiler("-cp", testOutput.path, "-opt:l:inline", "-opt-inline-from:**"))(bCode)
compileString(newCompiler("-cp", testOutput.path, "-opt:inline:**"))(bCode)
}

def readClass(file: String) = {
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/indyLambdaKinds/Test_2.scala
Expand Up @@ -3,7 +3,7 @@ import reflect.internal.util._

object Test extends DirectTest {

override def extraSettings: String = s"-usejavacp -cp ${testOutput.path} -opt:l:inline -opt-inline-from:** -Yopt-log-inline _"
override def extraSettings: String = s"-usejavacp -cp ${testOutput.path} -opt:inline:** -Yopt-log-inline _"

override def code = """object Main {
@noinline def t1a(a: A_1) = a.a(): @inline
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/inline-stack-map-frames/A_1.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
class A {
@noinline final def b: B = null
@inline final def a: A = b
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/inline-stack-map-frames/Test_2.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
class C {
def t(a: A): AnyRef = {
// a.a is inlined, resulting in a.b, which has return type B
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/noInlineUnknownIndy/Test.scala
Expand Up @@ -11,7 +11,7 @@ object Test extends DirectTest {

override def extraSettings = {
val classpath = List(sys.props("partest.lib"), testOutput.path).mkString(sys.props("path.separator"))
s"-cp $classpath -opt:l:inline -opt-inline-from:** -Yopt-inline-heuristics:everything -opt-warnings:_"
s"-cp $classpath -opt:inline:** -Yopt-inline-heuristics:everything -Wopt"
}

override def show(): Unit = {
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/repl-inline.scala
Expand Up @@ -16,7 +16,7 @@ assert(h == "h", h)
def main(args: Array[String]): Unit = {
def test(f: Settings => Unit): Unit = {
val settings = new Settings()
settings.processArgumentString("-opt:l:inline -opt-inline-from:** -opt-warnings")
settings.processArgumentString("-opt:inline:** -Wopt")
f(settings)
settings.usejavacp.value = true
val repl = new interpreter.IMain(settings, new ReplReporterImpl(settings))
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/synchronized.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
//
/*
* filter: optimizer warnings;
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t11255/A_1.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
class K(val f: Int => Int) extends Serializable
class A {
@inline final def f = new K(x => x + 1)
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t11255/Test_2.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
object Test {
def serializeDeserialize(obj: Object): Object = {
import java.io._
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t2106.scala
@@ -1,4 +1,4 @@
// scalac: -opt-warnings -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:** -Wopt
//
class A extends Cloneable {
@inline final def foo = clone()
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t3509.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
//
object Test {

Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t3569.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
//
object Test {
final val bippy1 = 1
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t4285.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
//
import scala.tools.partest.ReplTest
object Test extends ReplTest {
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t4935.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
//
object Test extends App {
for (i <- 0 to 1) {
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t5789.scala
Expand Up @@ -4,7 +4,7 @@ import scala.tools.partest.{Lambdaless, ReplTest}


object Test extends ReplTest with Lambdaless {
override def extraSettings = "-opt:l:inline -opt-inline-from:**"
override def extraSettings = "-opt:inline:**"
def code = """
val n = 2
() => n
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t6102.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:** -Werror
// scalac: -opt:inline:** -Werror
//
// scala/bug#6102 Wrong bytecode in lazyval + no-op finally clause

Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t6188.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
//
// scala/bug#6188 Optimizer incorrectly removes method invocations containing throw expressions

Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t7459b-optimize.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
//
class LM {
class Node[B1]
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t7582/InlineHolder_2.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:** -opt-warnings
// scalac: -opt:inline:** -Wopt -Werror
package p1 {
object InlineHolder {
@inline def inlinable = (p1.PackageProtectedJava_1.protectedMethod(): @noinline) + 1
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t7582b/InlineHolder_2.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:** -opt-warnings
// scalac: -opt:inline:** -Wopt
package p1 {
object InlineHolder {
@inline def inlinable = (p1.PackageProtectedJava_1.protectedMethod(): @noinline) + 1
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t8601-closure-elim.scala
@@ -1,4 +1,4 @@
// scalac: -Ydelambdafy:method -opt:l:inline -opt-inline-from:**
// scalac: -Ydelambdafy:method -opt:inline:**
//
import scala.tools.partest.BytecodeTest
import scala.tools.testkit.ASMConverters.instructionsFromMethod
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t8601.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
//
object Test {
def idiv(x: Int): Unit = x / 0
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t8601b.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
//
object Test {
def len(x: Array[String]): Unit = x.length
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t8601c.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
//
object Test {
def loadField(x: scala.runtime.IntRef): Unit = x.elem
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t8601d.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
//
object Test {
def monitor(x: AnyRef): Unit = {x.synchronized(()); ()}
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t8601e/Test.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
class C {
def foo: Unit = {StaticInit.fld}
}
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t9003.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
//
object Single {
var i = 0
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t9403/C_1.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
package p
class C {
@inline final def f(x: Int): Long = 10L / (if (x < 0) -2 else 2)
Expand Down
2 changes: 1 addition & 1 deletion test/files/run/t9403/Test_2.scala
@@ -1,4 +1,4 @@
// scalac: -opt:l:inline -opt-inline-from:**
// scalac: -opt:inline:**
import p.C
import scala.tools.asm.Opcodes
import scala.tools.partest.BytecodeTest
Expand Down
Expand Up @@ -14,7 +14,7 @@ import scala.tools.testkit.BytecodeTesting._
class InnerClassAttributeTest extends BytecodeTesting {
import compiler._

val optCompiler = cached("optCompiler", () => newCompiler(extraArgs = "-opt:l:inline -opt-inline-from:**"))
val optCompiler = cached("optCompiler", () => newCompiler(extraArgs = "-opt:inline:**"))

@Test
def javaInnerClassInGenericSignatureOnly(): Unit = {
Expand Down
Expand Up @@ -11,7 +11,7 @@ import scala.tools.testkit.BytecodeTesting._

@RunWith(classOf[JUnit4])
class OptimizedBytecodeTest extends BytecodeTesting {
override def compilerArgs = "-opt:l:inline -opt-inline-from:** -opt-warnings"
override def compilerArgs = "-opt:inline:** -Wopt"
import compiler._

@Test
Expand Down
Expand Up @@ -17,7 +17,7 @@ import scala.tools.testkit.BytecodeTesting._
*/
@RunWith(classOf[JUnit4])
class BoxUnboxAndInlineTest extends BytecodeTesting {
override def compilerArgs = "-opt:l:inline -opt-inline-from:**/*"
override def compilerArgs = "-opt:inline:**/*"
import compiler._

// Was crashing in 2.13.x once https://github.com/scala/scala/pull/9433 was merged in.
Expand Down
Expand Up @@ -13,7 +13,7 @@ import scala.tools.testkit.BytecodeTesting._

@RunWith(classOf[JUnit4])
class ClosureOptimizerTest extends BytecodeTesting {
override def compilerArgs = "-opt:l:inline -opt-inline-from:** -opt-warnings:_"
override def compilerArgs = "-opt:inline:** -Wopt"
import compiler._

@Test
Expand Down
Expand Up @@ -18,7 +18,7 @@ class InlineInfoTest extends BytecodeTesting {
import compiler._
import global.genBCode.{bTypes, postProcessor}

override def compilerArgs = "-opt:l:inline -opt-inline-from:**"
override def compilerArgs = "-opt:inline:**"

compiler.keepPerRunCachesAfterRun(List(
JavaClearable.forMap(bTypes.classBTypeCache),
Expand Down
Expand Up @@ -15,7 +15,7 @@ import scala.tools.testkit.BytecodeTesting._
class InlineSourceMatcherTest extends BytecodeTesting {
import compiler._

override def compilerArgs = "-opt:l:inline -opt-warnings"
override def compilerArgs = "-opt:inline -Wopt"
def setInlineFrom(s: String): Unit = {
//global.settings.optInlineFrom.value = s.split(':').toList
global.settings.optChoices.inline.selections = s.split(':').toList
Expand Down
Expand Up @@ -11,8 +11,7 @@ import scala.tools.testkit.BytecodeTesting._

@RunWith(classOf[JUnit4])
class InlineWarningTest extends BytecodeTesting {
def optInline = "-opt:l:inline -opt-inline-from:**"
override def compilerArgs = s"$optInline -opt-warnings"
override def compilerArgs = "-opt:inline:** -Wopt"

import compiler._

Expand Down
Expand Up @@ -10,7 +10,7 @@ import scala.tools.testkit.BytecodeTesting._

@RunWith(classOf[JUnit4])
class InlinerSeparateCompilationTest {
val args = "-opt:l:inline -opt-inline-from:**"
val args = "-opt:inline:**"

@Test
def inlineMixedinMember(): Unit = {
Expand Down
Expand Up @@ -18,7 +18,7 @@ import scala.tools.testkit.BytecodeTesting._

@RunWith(classOf[JUnit4])
class InlinerTest extends BytecodeTesting {
override def compilerArgs = "-opt:l:inline -opt-inline-from:** -opt-warnings"
override def compilerArgs = "-opt:inline:** -Wopt"

val inlineOnlyCompiler = cached("inlineOnlyCompiler", () => newCompiler(extraArgs = "-opt:inline -opt-inline-from:**"))

Expand Down
Expand Up @@ -15,7 +15,7 @@ import PartialFunction.cond

@RunWith(classOf[JUnit4])
class PatmatBytecodeTest extends BytecodeTesting {
val optCompiler = cached("optCompiler", () => newCompiler(extraArgs = "-opt:l:inline -opt-inline-from:**"))
val optCompiler = cached("optCompiler", () => newCompiler(extraArgs = "-opt:inline:**"))

import compiler._

Expand Down

0 comments on commit dc9b785

Please sign in to comment.