Remove SelectorTable/SelectorInfo, simply store all selectors in the central IdentifierTable.
Rationale:
We currently have a separate table to unique ObjC selectors. Since I don't need all the instance data in IdentifierInfo, I thought this would save space (and make more sense conceptually).
It turns out the cost of having duplicate entries for unary selectors (i.e. names without colons) outweighs the cost difference between the IdentifierInfo & SelectorInfo structures. Here is the data:
Two tables:
*** Selector/Identifier Stats:
# Selectors/Identifiers: 51635
Bytes allocated:
1999824
One table:
*** Identifier Table Stats:
# Identifiers: 49500
Bytes allocated:
1990316
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@42139
91177308-0d34-0410-b5e6-
96231b3b80d8