This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH: Add objc-lang.c, further cleanup - TAKE TWO
- From: Stan Shebs <shebs at apple dot com>
- To: Ziemowit Laski <zlaski at apple dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Mon, 03 Dec 2001 16:04:32 -0800
- Subject: Re: PATCH: Add objc-lang.c, further cleanup - TAKE TWO
- References: <4FB0D56A-E845-11D5-92F8-0030658361CA@apple.com>
Ziemowit Laski wrote:
>
> This patch moves the lang_hooks machinery from objc-act.c to
> a newly-created objc-lang.c, for symmetry with other front-ends.
> Also, a real hash function (i.e., one based on the value rather
> than the address) has been provided for storing message
> selectors. Finally, typos and copyright notices have been
> corrected here and there.
>
> Per Neil's suggestion, I've removed the call to maybe_add_objc_tree_codes
> from c-lang.c and placed it in objc-act.c.
>
> Bootstrapped successfully on i686-pc-linux-gnu, no regressions.
> OK to commit?
This looks good, please go ahead and commit it. Might as well
take the opportunity to put the ^L on a line by itself:
! ^L/* Compute a hash value for a given method SEL_NAME. */
Incidentally, I understand the purpose of hash_func is to use
something more predictable than the raw address of a tree node,
but this does entail a speed hit. Did you ever get any numbers
for the effect on the processing of a large ObjC header, like
OS X or GNUstep Foundation.h or AppKit.h?
Stan