[PATCH] checking version of rtl flag access macros

John David Anglin dave@hiauly1.hia.nrc.ca
Wed May 8 12:03:00 GMT 2002


> There were conflicting descriptions of what this flag means, and I
> changed this comment to match the other descriptions; it's not for a
> branch itself, it's for an instruction in the delay slot of a branch.

In pa.md, it's definitely used for the branch itself.

> 
> ia64-unknown-linux-gnu.  It's possible that on some targets it's valid
> for a JUMP_INSN to be in the delay slot of a branch; I'll let people
> who are more knowledgeable about this than I am figure that out.

Yes, unconditional jumps can be in the delay slot of a branch on the pa.

> INSN in the comment refers to the expected rtx code.  The new macros
> RTL_FLAG_CHECK* take a list of rtx codes to compare against the code
> of the rtx argument of the macro.

I think "INSN" should be in lowercase, and some reference to the macro's
argument "RTX" should be in the comment.

Dave
-- 
J. David Anglin                                  dave.anglin@nrc.ca
National Research Council of Canada              (613) 990-0752 (FAX: 952-6605)



More information about the Gcc-patches mailing list