Merge pull request #11960 from dotty-staging/fix-11885b

Fix #11885: Always clone trees if position already set

Author: liufengyun

compiler/src/dotty/tools/dotc/ast/Positioned.scala

@@ -65,12 +65,11 @@ abstract class Positioned(implicit @constructorOnly src: SourceFile) extends Src
     if (span == mySpan) this
     else {
       val newpd: this.type =
-        if (mySpan.isSynthetic) {
-          if (!mySpan.exists && span.exists)
-            envelope(source, span.startPos) // fill in children spans
+        if !mySpan.exists then
+          if span.exists then envelope(source, span.startPos) // fill in children spans
           this
-        }
-        else cloneIn(source)
+        else
+          cloneIn(source)
       newpd.span = span
       newpd
     }

compiler/test/dotty/tools/dotc/IdempotencyTests.scala

@@ -18,9 +18,8 @@ class IdempotencyTests {
   import IdempotencyTests._
   import CompilationTest.aggregateTests
 
-  // Flaky test on Windows
-  // https://github.com/lampepfl/dotty/issues/11885
-  val filter = FileFilter.exclude("i6507b.scala")
+  // ignore flaky tests
+  val filter = FileFilter.NoFilter
 
   @Category(Array(classOf[SlowTests]))
   @Test def idempotency: Unit = {