From 737ec374868178047b342a4a93152ca5fe80e58c Mon Sep 17 00:00:00 2001
From: Mikhail Glukhikh <mikhail.glukhikh@jetbrains.com>
Date: Wed, 21 Mar 2018 12:53:37 +0300
Subject: [PATCH] FIR visibility rendering: unknown -> public with question

---
 .../org/jetbrains/kotlin/fir/FirRenderer.kt   | 14 ++++++--
 .../fir/rawBuilder/declarations/F.txt         |  4 +--
 .../declarations/NestedOfAliasedType.txt      | 10 +++---
 .../declarations/NestedSuperType.txt          | 10 +++---
 .../rawBuilder/declarations/complexTypes.txt  | 10 +++---
 .../rawBuilder/declarations/derivedClass.txt  | 10 +++---
 .../fir/rawBuilder/declarations/enums.txt     | 36 +++++++++----------
 .../fir/rawBuilder/declarations/enums2.txt    | 20 +++++------
 .../rawBuilder/declarations/functionTypes.txt |  6 ++--
 .../declarations/genericFunctions.txt         |  8 ++---
 .../rawBuilder/declarations/nestedClass.txt   | 12 +++----
 .../rawBuilder/declarations/simpleClass.txt   | 26 +++++++-------
 .../fir/rawBuilder/declarations/simpleFun.txt |  4 +--
 .../declarations/simpleTypeAlias.txt          |  6 ++--
 .../declarations/typeAliasWithGeneric.txt     |  8 ++---
 .../declarations/typeParameterVsNested.txt    | 22 ++++++------
 .../declarations/typeParameters.txt           | 18 +++++-----
 compiler/testData/fir/resolve/F.txt           |  4 +--
 .../fir/resolve/NestedOfAliasedType.txt       | 10 +++---
 .../testData/fir/resolve/NestedSuperType.txt  | 10 +++---
 .../fir/resolve/TwoDeclarationsInSameFile.txt |  4 +--
 .../fir/resolve/multifile/NestedSuperType.txt |  6 ++--
 .../resolve/multifile/simpleAliasedImport.txt |  2 +-
 .../fir/resolve/multifile/simpleImport.txt    |  2 +-
 .../resolve/multifile/simpleImportNested.txt  |  2 +-
 .../resolve/multifile/simpleImportOuter.txt   |  2 +-
 compiler/testData/fir/resolve/simpleClass.txt | 26 +++++++-------
 .../testData/fir/resolve/simpleTypeAlias.txt  |  6 ++--
 28 files changed, 153 insertions(+), 145 deletions(-)

diff --git a/compiler/fir/tree/src/org/jetbrains/kotlin/fir/FirRenderer.kt b/compiler/fir/tree/src/org/jetbrains/kotlin/fir/FirRenderer.kt
index d5a61f26489..e67315d336b 100644
--- a/compiler/fir/tree/src/org/jetbrains/kotlin/fir/FirRenderer.kt
+++ b/compiler/fir/tree/src/org/jetbrains/kotlin/fir/FirRenderer.kt
@@ -5,6 +5,8 @@
 
 package org.jetbrains.kotlin.fir
 
+import org.jetbrains.kotlin.descriptors.Visibilities
+import org.jetbrains.kotlin.descriptors.Visibility
 import org.jetbrains.kotlin.fir.declarations.*
 import org.jetbrains.kotlin.fir.expressions.*
 import org.jetbrains.kotlin.fir.types.*
@@ -98,6 +100,12 @@ class FirRenderer(builder: StringBuilder) : FirVisitorVoid() {
         callableMember.returnType.accept(this)
     }
 
+    private fun Visibility.asString() =
+        when (this) {
+            Visibilities.UNKNOWN -> "public?"
+            else -> toString()
+        }
+
     override fun visitMemberDeclaration(memberDeclaration: FirMemberDeclaration) {
         memberDeclaration.annotations.renderAnnotations()
         if (memberDeclaration.typeParameters.isNotEmpty()) {
@@ -105,7 +113,7 @@ class FirRenderer(builder: StringBuilder) : FirVisitorVoid() {
             memberDeclaration.typeParameters.renderSeparated()
             print("> ")
         }
-        print(memberDeclaration.visibility.toString() + " " + memberDeclaration.modality.name.toLowerCase() + " ")
+        print(memberDeclaration.visibility.asString() + " " + memberDeclaration.modality.name.toLowerCase() + " ")
         if (memberDeclaration is FirCallableMember && memberDeclaration.isOverride) {
             print("override ")
         }
@@ -217,7 +225,7 @@ class FirRenderer(builder: StringBuilder) : FirVisitorVoid() {
 
     override fun visitConstructor(constructor: FirConstructor) {
         constructor.annotations.renderAnnotations()
-        print(constructor.visibility.toString() + " constructor")
+        print(constructor.visibility.asString() + " constructor")
         constructor.valueParameters.renderParameters()
         constructor.delegatedConstructor?.accept(this)
         constructor.body?.accept(this)
@@ -228,7 +236,7 @@ class FirRenderer(builder: StringBuilder) : FirVisitorVoid() {
 
     override fun visitPropertyAccessor(propertyAccessor: FirPropertyAccessor) {
         propertyAccessor.annotations.renderAnnotations()
-        print(propertyAccessor.visibility.toString() + " ")
+        print(propertyAccessor.visibility.asString() + " ")
         print(if (propertyAccessor.isGetter) "get" else "set")
         propertyAccessor.valueParameters.renderParameters()
         print(": ")
diff --git a/compiler/testData/fir/rawBuilder/declarations/F.txt b/compiler/testData/fir/rawBuilder/declarations/F.txt
index 0e5f8855f41..30143e6164a 100644
--- a/compiler/testData/fir/rawBuilder/declarations/F.txt
+++ b/compiler/testData/fir/rawBuilder/declarations/F.txt
@@ -1,5 +1,5 @@
 FILE: F.kt
-    unknown open class A() {
+    public? open class A() {
     }
-    unknown final class B() : A {
+    public? final class B() : A {
     }
diff --git a/compiler/testData/fir/rawBuilder/declarations/NestedOfAliasedType.txt b/compiler/testData/fir/rawBuilder/declarations/NestedOfAliasedType.txt
index 11ea929a8c6..3e0ec055264 100644
--- a/compiler/testData/fir/rawBuilder/declarations/NestedOfAliasedType.txt
+++ b/compiler/testData/fir/rawBuilder/declarations/NestedOfAliasedType.txt
@@ -1,12 +1,12 @@
 FILE: NestedOfAliasedType.kt
-    unknown abstract class A() {
-        unknown abstract class Nested() {
+    public? abstract class A() {
+        public? abstract class Nested() {
         }
 
     }
-    unknown final typealias TA = A
-    unknown final class B() : TA {
-        unknown final class NestedInB() : Nested {
+    public? final typealias TA = A
+    public? final class B() : TA {
+        public? final class NestedInB() : Nested {
         }
 
     }
diff --git a/compiler/testData/fir/rawBuilder/declarations/NestedSuperType.txt b/compiler/testData/fir/rawBuilder/declarations/NestedSuperType.txt
index 7dd6d52344b..0c91eb38958 100644
--- a/compiler/testData/fir/rawBuilder/declarations/NestedSuperType.txt
+++ b/compiler/testData/fir/rawBuilder/declarations/NestedSuperType.txt
@@ -1,14 +1,14 @@
 FILE: NestedSuperType.kt
-    unknown abstract class My() {
-        unknown abstract class NestedOne() : My {
-            unknown abstract class NestedTwo() : NestedOne {
+    public? abstract class My() {
+        public? abstract class NestedOne() : My {
+            public? abstract class NestedTwo() : NestedOne {
             }
 
         }
 
     }
-    unknown final class Your() : My {
-        unknown final class NestedThree() : NestedOne {
+    public? final class Your() : My {
+        public? final class NestedThree() : NestedOne {
         }
 
     }
diff --git a/compiler/testData/fir/rawBuilder/declarations/complexTypes.txt b/compiler/testData/fir/rawBuilder/declarations/complexTypes.txt
index e0cb9666f55..3d5dee9339b 100644
--- a/compiler/testData/fir/rawBuilder/declarations/complexTypes.txt
+++ b/compiler/testData/fir/rawBuilder/declarations/complexTypes.txt
@@ -1,11 +1,11 @@
 FILE: complexTypes.kt
-    <T, out S> unknown final class C() {
-        <R, in P> unknown final class D(inner) {
+    <T, out S> public? final class C() {
+        <R, in P> public? final class D(inner) {
         }
 
     }
-    unknown final interface Test() {
-        unknown final property x(val): a.b.C<out CharSequence, *>.D<in List<*>, *>
-            unknown get(): a.b.C<out CharSequence, *>.D<in List<*>, *>
+    public? final interface Test() {
+        public? final property x(val): a.b.C<out CharSequence, *>.D<in List<*>, *>
+            public? get(): a.b.C<out CharSequence, *>.D<in List<*>, *>
 
     }
diff --git a/compiler/testData/fir/rawBuilder/declarations/derivedClass.txt b/compiler/testData/fir/rawBuilder/declarations/derivedClass.txt
index 6657735e32c..2c464f20c50 100644
--- a/compiler/testData/fir/rawBuilder/declarations/derivedClass.txt
+++ b/compiler/testData/fir/rawBuilder/declarations/derivedClass.txt
@@ -1,12 +1,12 @@
 FILE: derivedClass.kt
-    <T> unknown open class Base() {
-        unknown constructor(x: T)
+    <T> public? open class Base() {
+        public? constructor(x: T)
 
     }
-    <T : Any> unknown final class Derived() : Base<T> {
-        unknown constructor(x: T)STUB
+    <T : Any> public? final class Derived() : Base<T> {
+        public? constructor(x: T)STUB
 
     }
-    <T : Any> unknown final function create(x: T): Derived<T> {
+    <T : Any> public? final function create(x: T): Derived<T> {
         STUB
     }
diff --git a/compiler/testData/fir/rawBuilder/declarations/enums.txt b/compiler/testData/fir/rawBuilder/declarations/enums.txt
index faa19d4b8d4..c7e7086593c 100644
--- a/compiler/testData/fir/rawBuilder/declarations/enums.txt
+++ b/compiler/testData/fir/rawBuilder/declarations/enums.txt
@@ -1,44 +1,44 @@
 FILE: enums.kt
-    unknown final enum class Order() {
-        unknown final enum entry FIRST() {
+    public? final enum class Order() {
+        public? final enum entry FIRST() {
         }
 
-        unknown final enum entry SECOND() {
+        public? final enum entry SECOND() {
         }
 
-        unknown final enum entry THIRD() {
+        public? final enum entry THIRD() {
         }
 
     }
-    unknown final enum class Planet() {
-        unknown constructor(m: Double, r: Double)
+    public? final enum class Planet() {
+        public? constructor(m: Double, r: Double)
 
-        unknown final enum entry MERCURY() : Planet {
-            unknown final override function sayHello(): <implicit> {
+        public? final enum entry MERCURY() : Planet {
+            public? final override function sayHello(): <implicit> {
             }
 
         }
 
-        unknown final enum entry VENERA() : Planet {
-            unknown final override function sayHello(): <implicit> {
+        public? final enum entry VENERA() : Planet {
+            public? final override function sayHello(): <implicit> {
             }
 
         }
 
-        unknown final enum entry EARTH() : Planet {
-            unknown final override function sayHello(): <implicit> {
+        public? final enum entry EARTH() : Planet {
+            public? final override function sayHello(): <implicit> {
             }
 
         }
 
-        unknown final property g(val): Double = STUB
-            unknown get(): Double
+        public? final property g(val): Double = STUB
+            public? get(): Double
 
-        unknown abstract function sayHello(): <implicit>
+        public? abstract function sayHello(): <implicit>
 
-        unknown final object Companion(companion) {
-            unknown final const property G(val): <implicit> = STUB
-                unknown get(): <implicit>
+        public? final object Companion(companion) {
+            public? final const property G(val): <implicit> = STUB
+                public? get(): <implicit>
 
         }
 
diff --git a/compiler/testData/fir/rawBuilder/declarations/enums2.txt b/compiler/testData/fir/rawBuilder/declarations/enums2.txt
index 42af278384a..780b0fd40b8 100644
--- a/compiler/testData/fir/rawBuilder/declarations/enums2.txt
+++ b/compiler/testData/fir/rawBuilder/declarations/enums2.txt
@@ -1,27 +1,27 @@
 FILE: enums2.kt
-    unknown final interface Some() {
+    public? final interface Some() {
     }
-    unknown final object O1() : Some {
+    public? final object O1() : Some {
     }
-    unknown final object O2() : Some {
+    public? final object O2() : Some {
     }
-    unknown final enum class SomeEnum() {
-        unknown constructor(x: Some)
+    public? final enum class SomeEnum() {
+        public? constructor(x: Some)
 
-        unknown final enum entry FIRST() : SomeEnum {
-            unknown final override function check(y: Some): Boolean {
+        public? final enum entry FIRST() : SomeEnum {
+            public? final override function check(y: Some): Boolean {
                 STUB
             }
 
         }
 
-        unknown final enum entry SECOND() : SomeEnum {
-            unknown final override function check(y: Some): Boolean {
+        public? final enum entry SECOND() : SomeEnum {
+            public? final override function check(y: Some): Boolean {
                 STUB
             }
 
         }
 
-        unknown abstract function check(y: Some): Boolean
+        public? abstract function check(y: Some): Boolean
 
     }
diff --git a/compiler/testData/fir/rawBuilder/declarations/functionTypes.txt b/compiler/testData/fir/rawBuilder/declarations/functionTypes.txt
index fdca19abceb..ddaa3f01c7a 100644
--- a/compiler/testData/fir/rawBuilder/declarations/functionTypes.txt
+++ b/compiler/testData/fir/rawBuilder/declarations/functionTypes.txt
@@ -1,9 +1,9 @@
 FILE: functionTypes.kt
-    <T> unknown final function simpleRun(f: ( (<no name provided>: T): Unit )): Unit {
+    <T> public? final function simpleRun(f: ( (<no name provided>: T): Unit )): Unit {
         STUB
     }
-    <T, R> unknown final function simpleMapList<T>.(f: ( (<no name provided>: T): R )): R {
+    <T, R> public? final function simpleMapList<T>.(f: ( (<no name provided>: T): R )): R {
     }
-    <T> unknown final function simpleWith(t: T, f: ( T.(): Unit )): Unit {
+    <T> public? final function simpleWith(t: T, f: ( T.(): Unit )): Unit {
         STUB
     }
diff --git a/compiler/testData/fir/rawBuilder/declarations/genericFunctions.txt b/compiler/testData/fir/rawBuilder/declarations/genericFunctions.txt
index b40ac57ea60..a70e7d8e334 100644
--- a/compiler/testData/fir/rawBuilder/declarations/genericFunctions.txt
+++ b/compiler/testData/fir/rawBuilder/declarations/genericFunctions.txt
@@ -1,10 +1,10 @@
 FILE: genericFunctions.kt
-    unknown final interface Any() {
+    public? final interface Any() {
     }
-    <T : Any> unknown final inline function safeAsAny.(): T? {
+    <T : Any> public? final inline function safeAsAny.(): T? {
         STUB
     }
-    unknown abstract class Summator() {
-        <T> unknown abstract function plus(first: T, second: T): T
+    public? abstract class Summator() {
+        <T> public? abstract function plus(first: T, second: T): T
 
     }
diff --git a/compiler/testData/fir/rawBuilder/declarations/nestedClass.txt b/compiler/testData/fir/rawBuilder/declarations/nestedClass.txt
index 77100615d34..801754239e9 100644
--- a/compiler/testData/fir/rawBuilder/declarations/nestedClass.txt
+++ b/compiler/testData/fir/rawBuilder/declarations/nestedClass.txt
@@ -1,15 +1,15 @@
 FILE: nestedClass.kt
-    unknown abstract class Base() {
-        unknown constructor(s: String)
+    public? abstract class Base() {
+        public? constructor(s: String)
 
     }
-    unknown final class Outer() {
-        unknown final class Derived() : Base {
-            unknown constructor(s: String)STUB
+    public? final class Outer() {
+        public? final class Derived() : Base {
+            public? constructor(s: String)STUB
 
         }
 
-        unknown final object Obj() : Base {
+        public? final object Obj() : Base {
         }
 
     }
diff --git a/compiler/testData/fir/rawBuilder/declarations/simpleClass.txt b/compiler/testData/fir/rawBuilder/declarations/simpleClass.txt
index 242b71899d6..6dab3348747 100644
--- a/compiler/testData/fir/rawBuilder/declarations/simpleClass.txt
+++ b/compiler/testData/fir/rawBuilder/declarations/simpleClass.txt
@@ -1,27 +1,27 @@
 FILE: simpleClass.kt
-    unknown final interface SomeInterface() {
-        unknown final function foo(x: Int, y: String): String
+    public? final interface SomeInterface() {
+        public? final function foo(x: Int, y: String): String
 
-        unknown final property bar(val): Boolean
-            unknown get(): Boolean
+        public? final property bar(val): Boolean
+            public? get(): Boolean
 
     }
-    unknown final class SomeClass() : SomeInterface {
+    public? final class SomeClass() : SomeInterface {
         private final property baz(val): <implicit> = STUB
-            unknown get(): <implicit>
+            public? get(): <implicit>
 
-        unknown final override function foo(x: Int, y: String): String {
+        public? final override function foo(x: Int, y: String): String {
         }
 
-        unknown final override property bar(var): Boolean
-            unknown get(): <implicit> {
+        public? final override property bar(var): Boolean
+            public? get(): <implicit> {
                 STUB
             }
-            unknown set(value: Boolean): <implicit> {
+            public? set(value: Boolean): <implicit> {
             }
 
-        unknown final property fau(var): Double
-            unknown get(): Double
-            unknown set(value: Double): R/kotlin.Unit/
+        public? final property fau(var): Double
+            public? get(): Double
+            public? set(value: Double): R/kotlin.Unit/
 
     }
diff --git a/compiler/testData/fir/rawBuilder/declarations/simpleFun.txt b/compiler/testData/fir/rawBuilder/declarations/simpleFun.txt
index f8b2fb8c0e5..6baf97e4021 100644
--- a/compiler/testData/fir/rawBuilder/declarations/simpleFun.txt
+++ b/compiler/testData/fir/rawBuilder/declarations/simpleFun.txt
@@ -1,3 +1,3 @@
 FILE: simpleFun.kt
-    unknown final function foo(): <implicit> {
-    }
\ No newline at end of file
+    public? final function foo(): <implicit> {
+    }
diff --git a/compiler/testData/fir/rawBuilder/declarations/simpleTypeAlias.txt b/compiler/testData/fir/rawBuilder/declarations/simpleTypeAlias.txt
index e73dd58a69f..092c1c9a097 100644
--- a/compiler/testData/fir/rawBuilder/declarations/simpleTypeAlias.txt
+++ b/compiler/testData/fir/rawBuilder/declarations/simpleTypeAlias.txt
@@ -1,6 +1,6 @@
 FILE: simpleTypeAlias.kt
-    unknown final interface B() {
+    public? final interface B() {
     }
-    unknown final typealias C = B
-    unknown final class D() : C {
+    public? final typealias C = B
+    public? final class D() : C {
     }
diff --git a/compiler/testData/fir/rawBuilder/declarations/typeAliasWithGeneric.txt b/compiler/testData/fir/rawBuilder/declarations/typeAliasWithGeneric.txt
index 27dca8cf45c..d69a2d6dfeb 100644
--- a/compiler/testData/fir/rawBuilder/declarations/typeAliasWithGeneric.txt
+++ b/compiler/testData/fir/rawBuilder/declarations/typeAliasWithGeneric.txt
@@ -1,8 +1,8 @@
 FILE: typeAliasWithGeneric.kt
-    unknown open class A() {
+    public? open class A() {
     }
-    <S, T : A> unknown final interface B() {
+    <S, T : A> public? final interface B() {
     }
-    unknown final typealias C = B<T, A>
-    unknown final class D() : C<A> {
+    public? final typealias C = B<T, A>
+    public? final class D() : C<A> {
     }
diff --git a/compiler/testData/fir/rawBuilder/declarations/typeParameterVsNested.txt b/compiler/testData/fir/rawBuilder/declarations/typeParameterVsNested.txt
index 0bb5993d725..e6144f07438 100644
--- a/compiler/testData/fir/rawBuilder/declarations/typeParameterVsNested.txt
+++ b/compiler/testData/fir/rawBuilder/declarations/typeParameterVsNested.txt
@@ -1,22 +1,22 @@
 FILE: typeParameterVsNested.kt
-    unknown final interface Some() {
+    public? final interface Some() {
     }
-    <T : Some> unknown abstract class My() {
-        unknown final class T(inner) {
+    <T : Some> public? abstract class My() {
+        public? final class T(inner) {
         }
 
-        unknown abstract property x(val): T
-            unknown get(): T
+        public? abstract property x(val): T
+            public? get(): T
 
-        unknown abstract function foo(arg: T): <implicit>
+        public? abstract function foo(arg: T): <implicit>
 
-        unknown abstract property y(val): My.T
-            unknown get(): My.T
+        public? abstract property y(val): My.T
+            public? get(): My.T
 
-        unknown abstract property z(val): test.My.T
-            unknown get(): test.My.T
+        public? abstract property z(val): test.My.T
+            public? get(): test.My.T
 
-        unknown final class Some() : T {
+        public? final class Some() : T {
         }
 
     }
diff --git a/compiler/testData/fir/rawBuilder/declarations/typeParameters.txt b/compiler/testData/fir/rawBuilder/declarations/typeParameters.txt
index 2b7383a4a02..026fdee2a79 100644
--- a/compiler/testData/fir/rawBuilder/declarations/typeParameters.txt
+++ b/compiler/testData/fir/rawBuilder/declarations/typeParameters.txt
@@ -1,20 +1,20 @@
 FILE: typeParameters.kt
-    <out T : Any> unknown final interface List() {
-        unknown final operator function get(index: Int): T
+    <out T : Any> public? final interface List() {
+        public? final operator function get(index: Int): T
 
-        unknown final infix function concat(other: List<T>): List<T>
+        public? final infix function concat(other: List<T>): List<T>
 
     }
-    unknown final typealias StringList = List<out String>
-    unknown final typealias AnyList = List<*>
-    <out T : Any> unknown abstract class AbstractList() : List<T> {
+    public? final typealias StringList = List<out String>
+    public? final typealias AnyList = List<*>
+    <out T : Any> public? abstract class AbstractList() : List<T> {
     }
-    unknown final class SomeList() : AbstractList<Int> {
-        unknown final override function get(index: Int): Int {
+    public? final class SomeList() : AbstractList<Int> {
+        public? final override function get(index: Int): Int {
             STUB
         }
 
-        unknown final override function concat(other: List<Int>): List<Int> {
+        public? final override function concat(other: List<Int>): List<Int> {
             STUB
         }
 
diff --git a/compiler/testData/fir/resolve/F.txt b/compiler/testData/fir/resolve/F.txt
index 98f13c208ea..0d25cec804b 100644
--- a/compiler/testData/fir/resolve/F.txt
+++ b/compiler/testData/fir/resolve/F.txt
@@ -1,5 +1,5 @@
 FILE: F.kt
-    (resolved) unknown open class A() {
+    (resolved) public? open class A() {
     }
-    (resolved) unknown final class B() : R/<root>.A/ {
+    (resolved) public? final class B() : R/<root>.A/ {
     }
diff --git a/compiler/testData/fir/resolve/NestedOfAliasedType.txt b/compiler/testData/fir/resolve/NestedOfAliasedType.txt
index 12893674cb1..9d7853c3e98 100644
--- a/compiler/testData/fir/resolve/NestedOfAliasedType.txt
+++ b/compiler/testData/fir/resolve/NestedOfAliasedType.txt
@@ -1,12 +1,12 @@
 FILE: NestedOfAliasedType.kt
-    (resolved) unknown abstract class A() {
-        unknown abstract class Nested() {
+    (resolved) public? abstract class A() {
+        public? abstract class Nested() {
         }
 
     }
-    (resolved) unknown final typealias TA = R/<root>.A/
-    (resolved) unknown final class B() : R/<root>.TA = <root>.A/ {
-        unknown final class NestedInB() : R/<root>.A.Nested/ {
+    (resolved) public? final typealias TA = R/<root>.A/
+    (resolved) public? final class B() : R/<root>.TA = <root>.A/ {
+        public? final class NestedInB() : R/<root>.A.Nested/ {
         }
 
     }
diff --git a/compiler/testData/fir/resolve/NestedSuperType.txt b/compiler/testData/fir/resolve/NestedSuperType.txt
index e1851206851..06ef67e622c 100644
--- a/compiler/testData/fir/resolve/NestedSuperType.txt
+++ b/compiler/testData/fir/resolve/NestedSuperType.txt
@@ -1,14 +1,14 @@
 FILE: NestedSuperType.kt
-    (resolved) unknown abstract class My() {
-        unknown abstract class NestedOne() : R/p.My/ {
-            unknown abstract class NestedTwo() : R/p.My.NestedOne/ {
+    (resolved) public? abstract class My() {
+        public? abstract class NestedOne() : R/p.My/ {
+            public? abstract class NestedTwo() : R/p.My.NestedOne/ {
             }
 
         }
 
     }
-    (resolved) unknown final class Your() : R/p.My/ {
-        unknown final class NestedThree() : R/p.My.NestedOne/ {
+    (resolved) public? final class Your() : R/p.My/ {
+        public? final class NestedThree() : R/p.My.NestedOne/ {
         }
 
     }
diff --git a/compiler/testData/fir/resolve/TwoDeclarationsInSameFile.txt b/compiler/testData/fir/resolve/TwoDeclarationsInSameFile.txt
index ef8869f366c..0ab93fdf803 100644
--- a/compiler/testData/fir/resolve/TwoDeclarationsInSameFile.txt
+++ b/compiler/testData/fir/resolve/TwoDeclarationsInSameFile.txt
@@ -1,5 +1,5 @@
 FILE: TwoDeclarationsInSameFile.kt
-    (resolved) unknown open class A() {
+    (resolved) public? open class A() {
     }
-    (resolved) unknown final class B() : R/p.A/ {
+    (resolved) public? final class B() : R/p.A/ {
     }
diff --git a/compiler/testData/fir/resolve/multifile/NestedSuperType.txt b/compiler/testData/fir/resolve/multifile/NestedSuperType.txt
index 21ef145caab..9742e83f54b 100644
--- a/compiler/testData/fir/resolve/multifile/NestedSuperType.txt
+++ b/compiler/testData/fir/resolve/multifile/NestedSuperType.txt
@@ -1,9 +1,9 @@
 FILE: NestedSuperType.kt
-    (resolved) unknown final class A() : R/b.B/ {
-        unknown final class NestedInA1() : R/b.B.NestedInB/ {
+    (resolved) public? final class A() : R/b.B/ {
+        public? final class NestedInA1() : R/b.B.NestedInB/ {
         }
 
-        unknown final class NestedInA2() : R/c.C.NestedInC/ {
+        public? final class NestedInA2() : R/c.C.NestedInC/ {
         }
 
     }
diff --git a/compiler/testData/fir/resolve/multifile/simpleAliasedImport.txt b/compiler/testData/fir/resolve/multifile/simpleAliasedImport.txt
index 06becd22d3e..adf3de7d4c7 100644
--- a/compiler/testData/fir/resolve/multifile/simpleAliasedImport.txt
+++ b/compiler/testData/fir/resolve/multifile/simpleAliasedImport.txt
@@ -1,3 +1,3 @@
 FILE: simpleAliasedImport.kt
-    (resolved) unknown final class YourClass() : R/b.MyClass/ {
+    (resolved) public? final class YourClass() : R/b.MyClass/ {
     }
diff --git a/compiler/testData/fir/resolve/multifile/simpleImport.txt b/compiler/testData/fir/resolve/multifile/simpleImport.txt
index 1fc18290145..6bcf374f37a 100644
--- a/compiler/testData/fir/resolve/multifile/simpleImport.txt
+++ b/compiler/testData/fir/resolve/multifile/simpleImport.txt
@@ -1,3 +1,3 @@
 FILE: simpleImport.kt
-    (resolved) unknown final class YourClass() : R/b.MyClass/ {
+    (resolved) public? final class YourClass() : R/b.MyClass/ {
     }
diff --git a/compiler/testData/fir/resolve/multifile/simpleImportNested.txt b/compiler/testData/fir/resolve/multifile/simpleImportNested.txt
index 6c16d2069ea..4d9e3c5dbdc 100644
--- a/compiler/testData/fir/resolve/multifile/simpleImportNested.txt
+++ b/compiler/testData/fir/resolve/multifile/simpleImportNested.txt
@@ -1,3 +1,3 @@
 FILE: simpleImportNested.kt
-    (resolved) unknown final class YourClass() : R/a.MyClass.MyNested/ {
+    (resolved) public? final class YourClass() : R/a.MyClass.MyNested/ {
     }
diff --git a/compiler/testData/fir/resolve/multifile/simpleImportOuter.txt b/compiler/testData/fir/resolve/multifile/simpleImportOuter.txt
index 01856c8d616..75de7c522af 100644
--- a/compiler/testData/fir/resolve/multifile/simpleImportOuter.txt
+++ b/compiler/testData/fir/resolve/multifile/simpleImportOuter.txt
@@ -1,3 +1,3 @@
 FILE: simpleImportOuter.kt
-    (resolved) unknown final class My() : R/a.Outer.Nested/ {
+    (resolved) public? final class My() : R/a.Outer.Nested/ {
     }
diff --git a/compiler/testData/fir/resolve/simpleClass.txt b/compiler/testData/fir/resolve/simpleClass.txt
index 742fe21fb66..631cce28a29 100644
--- a/compiler/testData/fir/resolve/simpleClass.txt
+++ b/compiler/testData/fir/resolve/simpleClass.txt
@@ -1,27 +1,27 @@
 FILE: simpleClass.kt
-    (resolved) unknown final interface SomeInterface() {
-        unknown final function foo(x: R/error: Failed to resolve qualified type/, y: R/error: Failed to resolve qualified type/): R/error: Failed to resolve qualified type/
+    (resolved) public? final interface SomeInterface() {
+        public? final function foo(x: R/error: Failed to resolve qualified type/, y: R/error: Failed to resolve qualified type/): R/error: Failed to resolve qualified type/
 
-        unknown final property bar(val): R/error: Failed to resolve qualified type/
-            unknown get(): R/error: Failed to resolve qualified type/
+        public? final property bar(val): R/error: Failed to resolve qualified type/
+            public? get(): R/error: Failed to resolve qualified type/
 
     }
-    (resolved) unknown final class SomeClass() : R/<root>.SomeInterface/ {
+    (resolved) public? final class SomeClass() : R/<root>.SomeInterface/ {
         private final property baz(val): R/error: Not supported: FirImplicitTypeImpl/ = STUB
-            unknown get(): R/error: Not supported: FirImplicitTypeImpl/
+            public? get(): R/error: Not supported: FirImplicitTypeImpl/
 
-        unknown final override function foo(x: R/error: Failed to resolve qualified type/, y: R/error: Failed to resolve qualified type/): R/error: Failed to resolve qualified type/ {
+        public? final override function foo(x: R/error: Failed to resolve qualified type/, y: R/error: Failed to resolve qualified type/): R/error: Failed to resolve qualified type/ {
         }
 
-        unknown final override property bar(var): R/error: Failed to resolve qualified type/
-            unknown get(): R/error: Not supported: FirImplicitTypeImpl/ {
+        public? final override property bar(var): R/error: Failed to resolve qualified type/
+            public? get(): R/error: Not supported: FirImplicitTypeImpl/ {
                 STUB
             }
-            unknown set(value: R/error: Failed to resolve qualified type/): R/error: Not supported: FirImplicitTypeImpl/ {
+            public? set(value: R/error: Failed to resolve qualified type/): R/error: Not supported: FirImplicitTypeImpl/ {
             }
 
-        unknown final property fau(var): R/error: Failed to resolve qualified type/
-            unknown get(): R/error: Failed to resolve qualified type/
-            unknown set(value: R/error: Failed to resolve qualified type/): R/kotlin.Unit/
+        public? final property fau(var): R/error: Failed to resolve qualified type/
+            public? get(): R/error: Failed to resolve qualified type/
+            public? set(value: R/error: Failed to resolve qualified type/): R/kotlin.Unit/
 
     }
diff --git a/compiler/testData/fir/resolve/simpleTypeAlias.txt b/compiler/testData/fir/resolve/simpleTypeAlias.txt
index cb8136e68c1..4071d40a18b 100644
--- a/compiler/testData/fir/resolve/simpleTypeAlias.txt
+++ b/compiler/testData/fir/resolve/simpleTypeAlias.txt
@@ -1,6 +1,6 @@
 FILE: simpleTypeAlias.kt
-    (resolved) unknown final interface B() {
+    (resolved) public? final interface B() {
     }
-    (resolved) unknown final typealias C = R/<root>.B/
-    (resolved) unknown final class D() : R/<root>.C = <root>.B/ {
+    (resolved) public? final typealias C = R/<root>.B/
+    (resolved) public? final class D() : R/<root>.C = <root>.B/ {
     }