Re: New and undocumented target hooks (cf. PR3386, a regression)

[Dorit Naishlos <DORIT at il dot ibm dot com>]

Here's the documentation with removal of the unused
target-hook/optab/tree-code.
Bootstrapped and tested on powerpc-darwin.

ok for mainline?

Changelog:

        * genopinit.c (vec_realign_store_optab): Initialization removed.
        * optabs.c (optab_for_tree_code): REALIGN_STORE_EXPR case removed.
        * optabs.h (optab_index): OTI_vec_realign_store Removed.
        (vec_realign_store_optab): Removed.
        * target-def.h (TARGET_VECTORIZE_BUILTIN_MASK_FOR_STORE): Removed.
        * target.h (builtin_mask_for_store): Removed.
        * tree.def (REALIGN_STORE_EXPR): Removed.
        * config/rs6000/rs6000.c (rs6000_builtin_mask_for_store): Removed.
        (altivec_builtin_mask_for_store): Removed.
        (altivec_init_builtins): Removed initialization of
        altivec_builtin_mask_for_store.
        * doc/tm.texi (TARGET_VECTORIZE_MISALIGNED_MEM_OK): Document target
hook.
        (TARGET_VECTORIZE_BUILTIN_MASK_FOR_LOAD): Document target hook.

Patch:

(See attached file: doc.patch)




                                                                                                                               
                      Richard Henderson                                                                                        
                      <rth@redhat.com>         To:       Dorit Naishlos/Haifa/IBM@IBMIL                                        
                                               cc:       Steven Bosscher <stevenb@suse.de>, gcc@gcc.gnu.org,                   
                      06/12/2004 22:52          gcc-patches@gcc.gnu.org                                                        
                                               Subject:  Re: New and undocumented target hooks (cf. PR3386, a regression)      
                                                                                                                               




On Mon, Dec 06, 2004 at 09:41:51PM +0200, Dorit Naishlos wrote:
> TARGET_VECTORIZE_BUILTIN_MASK_FOR_STORE is actually not yet used by the
> vectorizer - shall I remove its occurrences for now, or leave it as an
> indication of what is to come?

Unless you think this bit will get done for 4.0, take it out.


r~

Attachment: doc.patch
Description: Binary data