[PATCH] Don't create illegal memset with address space >= 256 in memcpyopt.
Manuel Jacob
me at manueljacob.de
Mon Jan 26 14:00:41 PST 2015
In http://reviews.llvm.org/D7175#113470, @compnerd wrote:
> This seems target dependent. Im not sure why, but, for example, SystemZ does permit it (or at the very least, continues to form the call. Perhaps move this behind a target specific check?
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.
http://reviews.llvm.org/D7175
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list