On Mon, Oct 8, 2012 at 10:18 AM, Jakub Jelinek <ja...@redhat.com> wrote: >> > I'm playing with a patch to expand the insns_with_changed_offsets >> > bitmap to an sbitmap, and will send a patch if this works better. >> >> Or make insns_with_changed_offsets a VEC of insns (or a pointer-set). > > Or use temporarily some rtx flag on the insns, from what I can see, > in_struct on *INSN is right now only used during scheduling and from reorg > till eoc, so for LRA sitting in between both scheduling passes it might > be possible to use that bit too.
AFAICT neither of these ideas will work because only insn UIDs are used when computing insns_with_changed_offsets. You'd need the actual insn for a VEC, pointer map or flag. Also, with a VEC or pointer map, it's difficult to union of the the insn_bitmap sets. The patch I have for this uses an sbitmap, it's posted in a new thread starting here: http://gcc.gnu.org/ml/gcc-patches/2012-10/msg00698.html Ciao! Steven