On 06/19/2018 08:11 AM, Tamar Christina wrote: > Hi All, > > Previously GCC's no-op detection could would consider something a no-op even > when the > mode change is not directly possible. This caused subregs that shouldn't be > removed > to be treated as a no-op and deleted. > > Regtested on armeb-none-eabi and no regressions. > Bootstrapped on arm-none-linux-gnueabihf and no issues. > > Ok for trunk? and for backport to GCC 8? > > Thanks, > Tamar > > gcc/ > 2018-06-19 Tamar Christina <tamar.christ...@arm.com> > > PR target/84711 > * rtlanal.c (set_noop_p): Constrain on mode change, > include hard-reg-set.h > OK. Though please include a testcase. I believe you had big-endian-subreg.c in the original target specific approach to fixing this problem. That'd be fine to re-use here.
jeff