• Nathan Bronson's avatar
    extend FOLLY_F14_PERTURB_INSERTION_ORDER to all F14FastMap/Set · 0c66e13e
    Nathan Bronson authored
    Summary:
    Previously the debug-build randomization of F14 iteration order
    was applied only to F14ValueMap/Set, F14NodeMap/Set, and F14FastMap/Set
    that uses the value storage strategy.  This extends the behavior to
    F14FastMap/Set that use the vector storage strategy, which are those
    instances where sizeof(value_type) >= 24.
    
    F14FastMap/Set using the vector storage strategy must move items
    to randomize, so this reordering will also expose cases that assume
    reference or iterator stability across multiple inserts without a call
    to .reserve().
    
    Reviewed By: yfeldblum
    
    Differential Revision: D13305818
    
    fbshipit-source-id: 178a1f7b707998728a0451af34269e735bf063f3
    0c66e13e
F14Table.cpp 2.25 KB