Use system hashcode for all names except SimpleTermNames

Keep start as a hash for SimpleTermNames because it distributed better.

1 files changed, 2 insertions, 4 deletions

diff --git a/compiler/src/dotty/tools/dotc/core/Names.scala b/compiler/src/dotty/tools/dotc/core/Names.scala
index dbaa338a2..9488aef61 100644
--- a/compiler/src/dotty/tools/dotc/core/Names.scala
+++ b/compiler/src/dotty/tools/dotc/core/Names.scala
@@ -227,13 +227,13 @@ object Names {
       !NameInfo.definesNewName(ownKind) && ownKind > kind && underlying.is(kind)
     }
 
-    override def hashCode: Int = start
-
     override protected[this] def newBuilder: Builder[Char, Name] = termNameBuilder
   }
 
   class SimpleTermName(val start: Int, val length: Int, @sharable private[Names] var next: SimpleTermName) extends TermName {
     // `next` is @sharable because it is only modified in the synchronized block of termName.
+    override def hashCode: Int = start
+
     override def toString =
       if (length == 0) "" else new String(chrs, start, length)
 
@@ -257,8 +257,6 @@ object Names {
     def without(kind: NameInfo.Kind): TypeName = toTermName.without(kind).toTypeName
     def is(kind: NameInfo.Kind) = toTermName.is(kind)
 
-    override def hashCode: Int = -start
-
     override protected[this] def newBuilder: Builder[Char, Name] =
       termNameBuilder.mapResult(_.toTypeName)