On Sat, Dec 04, 2004 at 07:01:16PM -0500, John David Anglin wrote: > + for (insn = next_insn (insn); > + insn && NOTE_P (insn); > + insn = next_insn (insn)); Please put a "continue;" on the next line. Otherwise it looks ok. r~