This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[PATCH] A few cleanups in tree-vect-*


Pretty obvious cleanups:
- couple typos
- couple instances of new-line before end of comment */
- some bad formatting
- make it explicit that the demotion/promotion vectorization functions deal
only with integers (in practice there's probably no functional change here,
cause if fp types are involved the tree-codes would be different (i.e. not
NOP/CONVERT_EXPR) and/or optabs support won't be available).
-  clarify how we set a vectype for a stmt in
vect_determine_vectorization_factor (added a comment, a couple asserts, and
removed redundant ifs).
- factor code into a new function - is_pattern_stmt_p

Bootstrapped on powerpc-linux.
Bootstrapped with vectorization enabled on i386-linux.
Tested on the vectorizer testcases on these two platforms.

ok to commit?

: ADDPATCH vectorizer (ssa, loops, non-algorithmic):

thanks,
dorit

ChangeLog:

2007-01-01  Dorit Nuzman  <dorit@il.ibm.com>
            Tehila Meyzels  <tehila@il.ibm.com>

        * tree-vectorizer.h (is_pattern_stmt_p): New.
        * tree-vect-analyze.c (vect_determine_vectorization_factor): Fix
        formatting (tabs instead of spaces). Cleanup and clarify setting
        of STMT_VINFO_VECTYPE. Call is_pattern_stmt_p.
        * tree-vect-transform.c (vect_get_vec_def_for_operand): Fix typo.
        (vectorizable_type_demotion): Check that types are integral.
        (vectorizable_type_promotion): Likewise.
        (vectorizable_store): Fix typo.  Eliminate new-line at end of
comments.

Patch:

(See attached file: cleanups.jan1.txt)

Attachment: cleanups.jan1.txt
Description: Text document


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]