[llvm-commits] Invalid instcombine transformation

[Anton Korobeynikov]

Hi, Chris

> The target *has* to support unaligned accesses.
Sorry, this is definitely unreal :) Especially in embedded world. And
not only there (think about sparc/solaris).

> I'd prefer to completely remove this from instcombine.  To do this, please verify that we are already doing the equivalent lowering in codegen (e.g. on x86 a 2 and 4-byte memset should turn into an unaligned store).  Then verify that disabling the instcombine xform doesn't cause any significant changes in codegen on multisource (or something else signifcant).
Ok.

-- 
With best regards, Anton Korobeynikov
Faculty of Mathematics and Mechanics, Saint Petersburg State University