public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Richard Henderson <rth@redhat.com>
To: Anatoly Sokolov <aesok@post.ru>
Cc: "Naveen H. S" <Naveen.S@kpitcummins.com>,
	       "law@redhat.com" <law@redhat.com>,
	       "gcc-patches@gcc.gnu.org" <gcc-patches@gcc.gnu.org>,
	       Prafulla Thakare <Prafulla.Thakare@kpitcummins.com>,
	       "kazu@codesourcery.com" <kazu@codesourcery.com>
Subject: Re: [PING]RE: [H8300] Hookize target macros
Date: Fri, 01 Apr 2011 17:28:00 -0000	[thread overview]
Message-ID: <4D960B32.7010602@redhat.com> (raw)
In-Reply-To: <307633434.20110401212547@post.ru>

On 04/01/2011 10:25 AM, Anatoly Sokolov wrote:
>         * config/h8300/h8300.c (print_operand_address): Rename to...
>         (h8300_print_operand_address): ...this. Make static. Adjust comments.
>         Call h8300_print_operand and h8300_print_operand_address instead of
>         print_operand and print_operand_address. Declare.
>         (print_operand): Renake to...
>         (h8300_print_operand): ...this. Make static. Adjust comments.
>         Call h8300_print_operand instead of print_operand. Declare.
>         (h8300_print_operand_punct_valid_p): Moved from h8300.h file.
>         (h8300_register_move_cost): Likewise.
>         (TARGET_PRINT_OPERAND_PUNCT_VALID_P, TARGET_PRINT_OPERAND
>         TARGET_PRINT_OPERAND_ADDRESS, TARGET_REGISTER_MOVE_COST): Define.
>         * config/h8300/h8300.h (PRINT_OPERAND_ADDRESS, PRINT_OPERAND
>         PRINT_OPERAND_PUNCT_VALID_P, REGISTER_MOVE_COST): Delete.
>         * config/h8300/h8300-protos.h (print_operand): Delete.
>         (print_operand_address): Delete

The new patch is ok too.


r~

      reply	other threads:[~2011-04-01 17:28 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-01  9:42 Naveen H. S
2010-09-03 15:06 ` Richard Henderson
2010-09-17 10:14   ` [PING]RE: " Naveen H. S
2011-04-01 17:25     ` Anatoly Sokolov
2011-04-01 17:28       ` Richard Henderson [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=4D960B32.7010602@redhat.com \
    --to=rth@redhat.com \
    --cc=Naveen.S@kpitcummins.com \
    --cc=Prafulla.Thakare@kpitcummins.com \
    --cc=aesok@post.ru \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=kazu@codesourcery.com \
    --cc=law@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).