On Wed, Mar 15, 2017 at 04:09:05PM +0000, Tamar Christina wrote: > This patch fixes a bug in the SIMD mov pattern where we were doing an > insert and leaving the rest of the vector in an undetermined state. This > could cause a bug if something else expects the other lanes to be 0. > > Semantically we wanted a mov here and not an insert. This is in line > with other patterns that use an fmov for the same reason, > e.g. aarch64_combinez*. > > Regression tested on aarch64-none-linux-gnu and no regressions.<br> > > OK for trunk?
This isn't technically a regression, but the bug would be subtle enough, and hard enough to debug that I think we should fix it for GCC 7. OK for trunk, where else does this need backported to? Thanks, James > gcc/ > 2017-03-15 Tamar Christina; <tamar.christ...@arm.com> > > * config/aarch64/aarch64-simd.md (*aarch64_simd_mov<mode>) > Change ins into fmov.
