public inbox for cgen@sourceware.org
 help / color / mirror / Atom feed
From: Nick Clifton <nickc@redhat.com>
To: Michael Meissner <bin-mail@the-meissners.org>
Cc: binutils@sources.redhat.com, cgen@sources.redhat.com
Subject: Re: Patches to move cgen files to C90 with prototypes
Date: Fri, 08 Aug 2003 09:27:00 -0000	[thread overview]
Message-ID: <m3vft8d0xs.fsf@redhat.com> (raw)
In-Reply-To: <20030807001021.GA31218@tiktok.the-meissners.org> (Michael Meissner's message of "Wed, 6 Aug 2003 20:10:21 -0400")

Hi Michael,

> (include/opcode directory)
>
> 2003-08-06  Michael Meissner  <gnu@the-meissners.org>
>
> 	* cgen.h: Remove PARAM macro usage in all prototypes.
> 	(CGEN_EXTRACT_INFO): Use void * instead of PTR.
> 	(cgen_print_fn): Ditto.
> 	(CGEN_HW_ENTRY): Ditto.
> 	(CGEN_MAYBE_MULTI_IFLD): Ditto.
> 	(struct cgen_insn): Ditto.
> 	(CGEN_CPU_TABLE): Ditto.
>
> (opcodes directory)
>
> 2003-08-06  Michael Meissner  <gnu@the-meissners.org>
>
> 	* cgen-ibld.in (insert_normal): Remove PARAMS macro.
> 	(insn_insn_normal): Ditto.
> 	(extract_normal): Ditto.
> 	(extract_insn_normal): Ditto.
> 	(put_insn_int_value): Ditto.
> 	(insert_1): Ditto.
> 	(fill_cache): Ditto.
> 	(extract_1): Ditto.
> 	(insert_1): Prototype definition.
> 	(insert_normal): Ditto.
> 	(insert_insn_normal): Ditto.
> 	(put_insn_int_value): Ditto.
> 	(fill_cache): Ditto.
> 	(extract_1): Ditto.
> 	(extract_normal): Ditto.
> 	(extract_insn_normal): Ditto.
>
> 	* cgen-dis.in (print_normal): Remove PARAMS macro.  Use void *
> 	instead of PTR.
> 	(print_address): Ditto.
> 	(print_keyword): Ditto.
> 	(print_insn_normal): Ditto.
> 	(print_insn): Ditto.
> 	(default_print_insn): Ditto.
> 	(read_insn): Ditto.
> 	(print_normal): Prototype definition.  Use void * instead of PTR.
> 	(print_address): Ditto.
> 	(print_keyword): Ditto.
> 	(print_insn_normal): Ditto.
> 	(read_insn): Ditto.
> 	(print_insn): Ditto.
> 	(default_print_insn): Ditto.
> 	(print_insn_@arch@): Ditto.
>
> 	* cgen-asm.in (parse_insn_normal): Remove PARAMS macro.
> 	(@arch@_cgen_build_insn_regex): Prototype definition.
> 	(parse_insn_normal): Ditto.
> 	(@arch@_cgen_assemble_insn): Ditto.
> 	(@arch@_cgen_asm_hash_keywords): Ditto.
>
> 	* cgen-dis.c (hash_insn_array): Remove PARAMS macro.
> 	(hash_insn_list): Ditto.
> 	(build_dis_hash_table): Ditto.
> 	(count_decodable_bits): Ditto.
> 	(add_insn_to_hash_chain): Ditto.
> 	(count_decodable_bits): Prototype definition.
> 	(add_insn_to_hash_chain): Ditto.
> 	(hash_insn_array): Ditto.
> 	(hash_insn_list): Ditto.
> 	(build_dis_hash_table): Ditto.
> 	(cgen_dis_lookup_insn): Ditto.
>
> 	* cgen-opc.c (hash_keyword_name): Remove PARAMS macro.
> 	(hash_keyword_value): Ditto.
> 	(build_keyword_hash_tables): Ditto.
> 	(cgen_keyword_lookup_name): Prototype definition.
> 	(cgen_keyword_lookup_value): Ditto.
> 	(cgen_keyword_add): Ditto.
> 	(cgen_keyword_search_init): Ditto.
> 	(cgen_keyword_search_next): Ditto.
> 	(hash_keyword_name): Ditto.
> 	(hash_keyword_value): Ditto.
> 	(build_keyword_hash_tables): Ditto.
> 	(cgen_hw_lookup_by_name): Ditto.
> 	(cgen_hw_lookup_by_num): Ditto.
> 	(cgen_operand_lookup_by_name): Ditto.
> 	(cgen_operand_lookup_by_num): Ditto.
> 	(cgen_insn_count): Ditto.
> 	(cgen_macro_insn_count): Ditto.
> 	(cgen_get_insn_value): Ditto.
> 	(cgen_put_insn_value): Ditto.
> 	(cgen_lookup_insn): Ditto.
> 	(cgen_get_insn_operands): Ditto.
> 	(cgen_lookup_get_insn_operands): Ditto.
> 	(cgen_set_signed_overflow_ok): Ditto.
> 	(cgen_clear_signed_overflow_ok): Ditto.
> 	(cgen_signed_overflow_ok_p): Ditto.
>
> 	* cgen-asm.c (hash_insn_array): Remove PARAMS macro.
> 	(hash_insn_list): Ditto.
> 	(build_asm_hash_table): Ditto.
> 	(cgen_set_parse_operand_fn): Prototype definition.
> 	(cgen_init_parse_operand): Ditto.
> 	(hash_insn_array): Ditto.
> 	(hash_insn_list): Ditto.
> 	(build_asm_hash_table): Ditto.
> 	(cgen_asm_lookup_insn): Ditto.
> 	(cgen_parse_keyword): Ditto.
> 	(cgen_parse_signed_integer): Ditto.
> 	(cgen_parse_unsigned_integer): Ditto.
> 	(cgen_parse_address): Ditto.
> 	(cgen_validate_signed_integer): Ditto.
> 	(cgen_validate_unsigned_integer): Ditto.

Approved - please apply.

Cheers
        Nick
        

      parent reply	other threads:[~2003-08-08  8:18 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-08-07  0:23 Michael Meissner
2003-08-07  0:44 ` Doug Evans
2003-08-07 13:52   ` Michael Meissner
2003-08-08  8:18     ` Nick Clifton
2003-08-08 11:49       ` Alan Modra
2003-08-08 12:15         ` Nick Clifton
2003-08-08 12:54           ` Frank Ch. Eigler
2003-08-08 13:07             ` Nick Clifton
2003-08-09  0:48               ` Alan Modra
2003-08-08  9:27 ` Nick Clifton [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=m3vft8d0xs.fsf@redhat.com \
    --to=nickc@redhat.com \
    --cc=bin-mail@the-meissners.org \
    --cc=binutils@sources.redhat.com \
    --cc=cgen@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).