[PATCH] D25419: Use unique_ptr in LLVMContextImpl's constant maps.

Justin Lebar via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 10 09:14:21 PDT 2016


jlebar marked an inline comment as done.
jlebar added inline comments.


================
Comment at: llvm/lib/IR/LLVMContextImpl.cpp:106
   for (auto &CDSConstant : CDSConstants)
     delete CDSConstant.second;
   CDSConstants.clear();
----------------
timshen wrote:
> Do you want to change this one as well?
Sure, but this one is complicated enough I'll put it in its own patch.


https://reviews.llvm.org/D25419





More information about the cfe-commits mailing list