Thank you for review, update and commit this patch set!

Jie

On 04/18/2011 10:04 AM, Richard Earnshaw wrote:

On Mon, 2010-10-11 at 15:44 +0800, Jie Zhang wrote:
This patch implements TARGET_BUILTIN_DECL for ARM. With the changes of
the previous two patches, this one is straightforward.

Is it OK?


Sorry for the long time reviewing this set of patches.  I've just
tweaked it to bring it up to the current code base and committed it.
It's largely unchanged from your submission apart from:

1) Updates to incorporate latest changes made by Richard Sandiford.
2) Minor tweak to simplyfy the iWMMXT builtins initialization.

R.

2011-04-18  Jie Zhang<j...@codesourcery.com>
        Richard Earnshaw<rearn...@arm.com>

        * arm.c (neon_builtin_type_bits): Remove.
        (typedef enum neon_builtin_mode): New.
        (T_MAX): Don't define.
        (typedef enum neon_builtin_datum): Remove bits, codes[],
        num_vars and base_fcode.  Add mode, code and fcode.
        (VAR1, VAR2, VAR3, VAR4, VAR5, VAR6, VAR7, VAR8, VAR9
        VAR10): Change accordingly.
        (neon_builtin_data[]): Change accordingly
        (arm_init_neon_builtins): Change accordingly.
        (neon_builtin_compare): Remove.
        (locate_neon_builtin_icode): Remove.
        (arm_expand_neon_builtin): Change accordingly.

        * arm.h (enum arm_builtins): Move to ...
        * arm.c (enum arm_builtins): ... here; and rearrange builtin code.

        * arm.c (arm_builtin_decl): Declare.
        (TARGET_BUILTIN_DECL): Define.
        (enum arm_builtins): Correct ARM_BUILTIN_MAX.
        (arm_builtin_decls[]): New.
        (arm_init_neon_builtins): Store builtin declarations in
        arm_builtin_decls[].
        (arm_init_tls_builtins): Likewise.
        (arm_init_iwmmxt_builtins): Likewise.  Refactor initialization code.
        (arm_builtin_decl): New.


Reply via email to