On Tue, 2014-08-12 at 15:08 -0600, Jeff Law wrote:
> On 08/06/14 11:19, David Malcolm wrote:
> > gcc/
> > * rtl.h (for_each_rtx_in_insn): New function.
> > * rtlanal.c (for_each_rtx_in_insn): Likewise.
> OK. Note that we're moving away from for_each_rtx... I haven't
> looked, but there'
On 08/14/14 15:36, Richard Sandiford wrote:
Right. I think they're held up on patch 40 (ironically the one that
conflicts with yours).
I think we could declare side effects in notes as invalid and add some
ENABLE_CHECKING bits to enforce that. With those in place, my concerns
around #40 from
David Malcolm writes:
> On Tue, 2014-08-12 at 15:08 -0600, Jeff Law wrote:
>> On 08/06/14 11:19, David Malcolm wrote:
>> > gcc/
>> >* rtl.h (for_each_rtx_in_insn): New function.
>> >* rtlanal.c (for_each_rtx_in_insn): Likewise.
>> OK. Note that we're moving away from for_each_rtx... I h
On Tue, 2014-08-12 at 15:08 -0600, Jeff Law wrote:
> On 08/06/14 11:19, David Malcolm wrote:
> > gcc/
> > * rtl.h (for_each_rtx_in_insn): New function.
> > * rtlanal.c (for_each_rtx_in_insn): Likewise.
> OK. Note that we're moving away from for_each_rtx... I haven't
> looked, but there'
On 08/06/14 11:19, David Malcolm wrote:
gcc/
* rtl.h (for_each_rtx_in_insn): New function.
* rtlanal.c (for_each_rtx_in_insn): Likewise.
OK. Note that we're moving away from for_each_rtx... I haven't
looked, but there's a reasonable chance we may not need it after Richard
S.'
gcc/
* rtl.h (for_each_rtx_in_insn): New function.
* rtlanal.c (for_each_rtx_in_insn): Likewise.
---
gcc/rtl.h | 1 +
gcc/rtlanal.c | 16
2 files changed, 17 insertions(+)
diff --git a/gcc/rtl.h b/gcc/rtl.h
index 0858230..3e37ed0 100644
--- a/gcc/rtl.h
+++ b/