erichkeane accepted this revision. erichkeane added a comment. This looks fine to change, and seems like a slight compile timeimprovement.
It seems that the only thing we use this for is .find, .erase, and operator[], all of which don't depend on iterator order, so this should not cause any problems with determinism. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118169/new/ https://reviews.llvm.org/D118169 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits