rjmccall added a comment. Looks good, but you missed updating equality/hashing for the new Kind field.
================ Comment at: lib/CodeGen/CodeGenTBAA.h:71 AccessType == Other.AccessType && Offset == Other.Offset; } ---------------- This needs to factor in the Kind. ================ Comment at: lib/CodeGen/CodeGenTBAA.h:205 DenseMapInfo<MDNode *>::getHashValue(Val.AccessType) ^ DenseMapInfo<uint64_t>::getHashValue(Val.Offset); } ---------------- This needs to factor in the Kind. https://reviews.llvm.org/D39008 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits