arphaman added a comment. LGTM, we can optimize later. It looks like you also moved `SymbolID` to a new file, this should be an NFC precommit.
CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54799/new/ https://reviews.llvm.org/D54799 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits