On Tue, Dec 3, 2013 at 4:41 AM, Jan Hubicka <hubi...@ucw.cz> wrote: >> Here is the updated patch. Tested on Linux/x86-64. It >> fixed git. OK to install? >> >> Thanks. >> >> -- >> H.J. >> --- >> gcc/ >> >> 2013-12-03 H.J. Lu <hongjiu...@intel.com> >> >> PR target/59363 >> * config/i386/i386.c (emit_memset): Adjust destination address >> after gen_strset. >> (expand_setmem_epilogue): Likewise. >> >> gcc/testsuite/ >> >> 2013-12-03 H.J. Lu <hongjiu...@intel.com> >> >> PR target/59363 >> * gcc.target/i386/pr59363.c: New file. > > Yes, this seems fine to me. As discussed previously, we probably want to make > strmov patterns use to match strset (I will need to re-check codegen on > targets > that does single memops) and then we will need similar update of aliasing > there, too. > Currently I assume we are fine becaue we use it only in expand_movmem epilogue > and on the way there we already cleared the alias offset on all code paths? >
I believe it is the case. I checked it in. Thanks. -- H.J.
