This is the mail archive of the binutils@sourceware.org mailing list for the binutils 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: [PATCH] New ia64 @slotcount pseudo func


>>> Douglas B Rupp <rupp@gnat.com> 28.10.09 21:24 >>>
>Submitted for comment as per previous discussions.

Why can't the debugger do the math that you do here? Unless Intel's
assembler is intending to also support @slotcount, I'm not certain it's a
good idea to add such to gas.

Also:

>+	      /* Calculate the number of instruction slots between the two
>+		 labels.  They are guaranteed to be in the same (.text)
>+		 section.  */
>+	      end = S_GET_VALUE (symbolPend);
>+	      beg = S_GET_VALUE (symbolPbeg);

Where is this "guarantee" being enforced? I think you mean "They are
supposed...", and need to check this really is the case.

Jan


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