Replace String.CompareOrdinal to string.Equals in KnownTypes

[lindexi]

Description

We can use string.Equals(string1, string2, StringComparison.Ordinal) to check for equality, see https://learn.microsoft.com/en-us/dotnet/csharp/how-to/compare-strings and https://learn.microsoft.com/en-us/dotnet/standard/base-types/best-practices-strings

Use the String.Compare and String.CompareTo methods to sort strings, not to check for equality.

Reference #6588

Cc @Kuldeep-MS

Customer Impact

Small performance improvements

Regression

None.

Testing

Just CI.

Risk

Low.

Microsoft Reviewers: Open in CodeFlow

[Symbai]

Curious why this is limited to KnownTypes only? A quick search here on Github shows that string.Compare is being used in >36 files. Wouldn't it make sense changing them all at once, so you don't end up with mixed coding styles?

[lindexi]

Thank you @Symbai

Please don't worry, I will continue to optimize the code. Small changes help code review, and changing too much code at once can lead to conflicts

[omariom]

For x86 string equality operator generates a bit more efficient code.

[lindexi]

Thank you @dipeshmsft

And I will update the other files soon @Symbai