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]

Re: [patch] Associate a name with a CODE_LABEL


Hi Catherine,

: This patch adds an alternate name field to a CODE_LABEL.
: If a CODE_LABEL has a name associated with it, use the
: name instead of an internal label.
: 
: Okay to commit?

Sorry to be picky but....

  * Is ASM_OUTPUT_ALTERNATIVE_LABEL_NAME a new target macro, and if so
    shouldn't it be documented, and a default definition provided ?

  * Why have you changed delete_unrefernced_labels() to not delete
    labels which have names, even if their use count is zero ?

  * Why does print_rtx() always dump out the alternative name field
    even if an alternative name has not been defined ?

Cheers
	Nick


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