Since referenced assembly identities can change (if they're an AssemblyBuilder), we have to postpone creating an AssemblyRef record until save time.

Note that we also take into account the possibility of having multiple assemblies with the same identity and collapse those into a single record. This is to support circular ref scenarios where you load a previous version of an assembly while generating a newer version (with the same identity). A future patch will enable this.

1 files changed, 21 insertions, 0 deletions

diff --git a/reflect/Metadata/Tables.cs b/reflect/Metadata/Tables.cs
index b6ba2586..c1977b19 100644
--- a/reflect/Metadata/Tables.cs
+++ b/reflect/Metadata/Tables.cs
@@ -1991,6 +1991,27 @@ namespace IKVM.Reflection.Metadata
 			internal int HashValue;
 		}
 
+		internal int FindOrAddRecord(Record rec)
+		{
+			for (int i = 0; i < rowCount; i++)
+			{
+				if (records[i].Name == rec.Name
+					&& records[i].MajorVersion == rec.MajorVersion
+					&& records[i].MinorVersion == rec.MinorVersion
+					&& records[i].BuildNumber == rec.BuildNumber
+					&& records[i].RevisionNumber == rec.RevisionNumber
+					&& records[i].Flags == rec.Flags
+					&& records[i].PublicKeyOrToken == rec.PublicKeyOrToken
+					&& records[i].Culture == rec.Culture
+					&& records[i].HashValue == rec.HashValue
+					)
+				{
+					return i + 1;
+				}
+			}
+			return AddRecord(rec);
+		}
+
 		internal override void Read(MetadataReader mr)
 		{
 			for (int i = 0; i < records.Length; i++)