This function

type AssemblyReference = 
    member x.SimpleAssemblyNameIs(name) = 

calls fileNameWithoutExtension which in turn calls checkPathForIllegalChars. However, x.SimpleAssemblyNameIs only uses the result of fileNameWithoutExtension to compare with a passed in Value name.

So there is no real reason for validating that fileNameWithoutExtension has illegal chars, since it will just fail to match the name.

This reduces the number of calls from

checkPathForIllegalCharsCalls : {contents = 947;}
fileNameWithoutExtensionCalls : {contents = 745;}

down to

checkPathForIllegalCharsCalls : {contents = 203;}
fileNameWithoutExtensionCalls : {contents = 745;}