Put the Crossgen2 assertion check in sync with actual intrinsic count (#44624)
authorTomáš Rylek <trylek@microsoft.com>
Fri, 13 Nov 2020 15:05:02 +0000 (16:05 +0100)
committerGitHub <noreply@github.com>
Fri, 13 Nov 2020 15:05:02 +0000 (07:05 -0800)
* Put the Crossgen2 assertion check in sync with actual intrinsic count

Recent removal of two intrinsics from the CorInfoIntrinsics enumeration
started tripping the assertion failure verifying the expected number
of intrinsics. I'm just updating the expected count.

* Remove the assertion per JanK's PR feedback

src/coreclr/src/tools/Common/JitInterface/CorInfoImpl.Intrinsics.cs

index 2f53520..bd6aaca 100644 (file)
@@ -111,9 +111,6 @@ namespace Internal.JitInterface
             table.Add(CorInfoIntrinsics.CORINFO_INTRINSIC_GetRawHandle, "DefaultConstructorOf", "System", "Activator");
             table.Add(CorInfoIntrinsics.CORINFO_INTRINSIC_GetRawHandle, "AllocatorOf", "System", "Activator");
 
-            // If this assert fails, make sure to add the new intrinsics to the table above and update the expected count below.
-            Debug.Assert((int)CorInfoIntrinsics.CORINFO_INTRINSIC_Count == 34, "Please update intrinsic hash table");
-
             return table;
         }