This replaces the C strings in the TypeSystem API with StringRefs. Also removes all the
StringRef locals that were previously used in the implementation as they are no longer necessary.
I probably should have commented on this. At the start of the function we already do the same check:
if (!enum_type || ConstString(name).IsEmpty()) return nullptr;
Still, thanks for the careful review :)