[PATCH] Don't create illegal memset with address space >= 256 in memcpyopt.

Matt Arsenault Matthew.Arsenault at amd.com
Mon Jan 26 14:15:50 PST 2015


On 01/26/2015 02:00 PM, Manuel Jacob wrote:
> I don't think this is target dependent.  See this assert<http://reviews.llvm.org/diffusion/L/browse/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp;227138$4800>.  It fails for any target if the address space is >= 256.  As I said on IRC, you probably tested with assertions disabled.
That's one of the places I was talking about. That assertion is a bug. 
It's an x86-ism that somehow infested itself there. The address space 
limit is much higher than that (the DataLayout enforces a limit of 24-bits)



More information about the llvm-commits mailing list