On Tue, 2014-08-12 at 15:22 -0600, Jeff Law wrote:
> On 08/06/14 11:19, David Malcolm wrote:
> > gcc/
> > 	* sel-sched-ir.h (BB_NOTE_LIST): struct sel_region_bb_info_def's
> > 	"note_list" field will eventually be an rtx_insn *.  To help with
> > 	transition, for now, convert from an access macro into a pair of
> > 	functions: BB_NOTE_LIST, returning an rtx_insn *, and...
> > 	(SET_BB_NOTE_LIST): New function, for use where BB_NOTE_LIST is
> > 	used as an lvalue.
> >
> > 	* sel-sched.c (create_block_for_bookkeeping): Update lvalue usage
> > 	of BB_NOTE_LIST to SET_BB_NOTE_LIST.
> >
> > 	* sel-sched-ir.c (init_bb): Likewise.
> > 	(sel_restore_notes): Likewise.
> > 	(move_bb_info): Likewise.
> > 	(BB_NOTE_LIST): New function, adding a checked cast to rtx_insn *.
> > 	(SET_BB_NOTE_LIST): New function.
> >
> > /
> > 	* rtx-classes-status.txt: Add SET_BB_NOTE_LIST.
> OK.
Thanks.

Fixed up the as_a_nullable to safe_as_a, and committed to trunk as
r214167, having verified bootstrap&regrtest on x86_64-unknown-linux-gnu
(Fedora 20) albeit in combination with patches 9-29 [1], and verified
that it builds standalone with 9 targets.

Am attaching what I committed.

FWIW these functions becomes a macro again in patch #170.


Dave

[1] as per https://gcc.gnu.org/ml/gcc-patches/2014-08/msg01420.html