Clean up
First version of NoStringConcatDiagnostic

Still detects all calls to the params version with a real array but
shouldn't
Rename TypeSymbolExtensions
Move all basic type checks to TypeSymbolExtensions
Extract code for overload checks
Add the missing checks for String.Concat
Fast exit for String.Concat check

Done without semantic model usage