[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