Skip to content
  • Steve Naroff's avatar
    · 2cd263ff
    Steve Naroff authored
    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
    
    llvm-svn: 42139
    2cd263ff
Loading