This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA, ARM] Fix line number data for PIC register setup code


On 12/20/2010 10:26 AM, Richard Earnshaw wrote:
>> +	      for (insn = seq; insn; insn = NEXT_INSN (insn))
>> +		if (INSN_P (insn))
>> +		  INSN_LOCATOR (insn) = prologue_locator;
>> +
> 
> Hmm, what if get_insns() doesn't return an INSN_P(), which is presumably
> what the test in the 'if' clause is for?  Won't it then abort in
> NEXT_INSN()?

No, it won't.  INSN_P does not include notes, for example.


r~


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]