timshen added inline comments. ================ Comment at: llvm/lib/IR/LLVMContextImpl.cpp:106 for (auto &CDSConstant : CDSConstants) delete CDSConstant.second; CDSConstants.clear(); ---------------- Do you want to change this one as well? https://reviews.llvm.org/D25419