[llvm-commits] [llvm] r158358 - in /llvm/trunk: include/llvm/Constants.h include/llvm/Instruction.h lib/Transforms/Scalar/Reassociate.cpp lib/VMCore/Constants.cpp lib/VMCore/Instruction.cpp test/Transforms/Reassociate/repeats.ll

Duncan Sands baldrick at free.fr
Tue Jun 12 13:20:30 PDT 2012


On 12/06/12 22:07, Duncan Sands wrote:
> Hi Matt, thanks for the great test case.  It seems to be a bug in SmallMap: it
> assumes that the key has POD type.  Alternatively, maybe the key is required to
> be a POD type but I don't see that documented anywhere.  Hopefully Stepan can
> clarify.  In the meantime I will change the code to use a std::map.

PS: Sorry, should have been: assumes that the value has POD type.  Maybe it also
assumes that the key has POD type, but here it is the value that is non-POD.

Ciao, Duncan.



More information about the llvm-commits mailing list