This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [ARC] RFA: Use new rtl iterators in arc600_corereg_hazard
- From: Joern Rennecke <joern dot rennecke at embecosm dot com>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>, Joern Rennecke <joern dot rennecke at embecosm dot com>, rdsandiford at googlemail dot com
- Date: Sun, 2 Nov 2014 18:41:12 +0000
- Subject: Re: [ARC] RFA: Use new rtl iterators in arc600_corereg_hazard
- Authentication-results: sourceware.org; auth=none
- References: <878uk4wjz9 dot fsf at googlemail dot com>
On 25 October 2014 10:56, Richard Sandiford <rdsandiford@googlemail.com> wrote:
> This is part of a series to remove uses of for_each_rtx from the ports.
>
> Tested by making sure there were no code changes for gcc.dg, gcc.c-torture
> and g++.dg for arc-elf. OK to install?
>
> Thanks,
> Richard
>
>
> gcc/
> * config/arc/arc.c (arc600_corereg_hazard_1): Delete.
> (arc600_corereg_hazard): Use FOR_EACH_SUBRTX.
OK.
> + FOR_EACH_SUBRTX (iter, array, PATTERN (pred), NONCONST)
I was wondering for a while what kind of (NON)CONST this was about...
but as I glean
from the source, as long as nobody packs an address with a side effect
into a (CONST (MEM (...)) -
which should never happen in the first place, no matter if the MEM
itself is really const - we
should be fine.