[PATCH] D11629: APValues and Constants and MaterializedTemporaryExpr need to have stable maps

David Majnemer david.majnemer at gmail.com
Thu Jul 30 00:10:16 PDT 2015


majnemer created this revision.
majnemer added a reviewer: rsmith.
majnemer added a subscriber: cfe-commits.

We risk iterator invalidation issues if we use DenseMap structures for
MaterializedTemporaryExprs.  Use a std::map to ensure that they don't
move around in memory.

This fixes PR24289.

http://reviews.llvm.org/D11629

Files:
  include/clang/AST/ASTContext.h
  lib/AST/ASTContext.cpp
  lib/CodeGen/CodeGenModule.h
  test/CodeGenCXX/PR24289.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D11629.30999.patch
Type: text/x-patch
Size: 11611 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150730/e376b43a/attachment.bin>


More information about the cfe-commits mailing list