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]

Re: [lno] Fix bootstrap on ia64



> The variable params is used everywhere, not only with
> BUILT_IN_build_mask_for_load.

Thanks.

BUILT_IN_build_mask_for_load was actually supposed to be removed all
together. The attached patch cleans up remaining occurrences of
BUILT_IN_build_mask_for_load, fixes expected testcase results on x86, and
brings over from mainline Paolo's patch to tree-vectorizer.c.

dorit

2004-10-03  Dorit Naishlos  <dorit@il.ibm.com>

        * tree-vectorizer.c (BUILT_IN_build_mask_for_load): Removed.

2004-10-03  Paolo Bonzini  <bonzini@gnu.org>

        * tree-vectorizer.c (vectorizable_operation): Fail unless
        the mode for the vector type is indeed a vector mode.

(See attached file: lnodiff.oct3)



|---------+----------------------------->
|         |           Andreas Schwab    |
|         |           <schwab@suse.de>  |
|         |           Sent by:          |
|         |           gcc-patches-owner@|
|         |           gcc.gnu.org       |
|         |                             |
|         |                             |
|         |           30/09/2004 12:10  |
|---------+----------------------------->
  >----------------------------------------------------------------------------------------------------------------------------|
  |                                                                                                                            |
  |       To:       gcc-patches@gcc.gnu.org                                                                                    |
  |       cc:                                                                                                                  |
  |       Subject:  [lno] Fix bootstrap on ia64                                                                                |
  >----------------------------------------------------------------------------------------------------------------------------|




The variable params is used everywhere, not only with
BUILT_IN_build_mask_for_load.  I've also moved the declaration down to the
smallest enclosing block.  Committed as obvious.

Andreas.


2004-09-30  Andreas Schwab  <schwab@suse.de>

             * tree-vectorizer.c (vectorizable_load): Always declare
params.

--- gcc/tree-vectorizer.c            30 Sep 2004 11:10:32 +0200
1.1.2.69
+++ tree-vectorizer.c          30 Sep 2004 11:17:12 +0200
@@ -2687,9 +2687,6 @@ vectorizable_load (tree stmt, block_stmt
       tree msq_init;
       tree msq, lsq;
       tree dataref_ptr;
-#ifdef BUILT_IN_build_mask_for_load
-      tree params;
-#endif

       /* <1> Create msq_init = *(floor(p1)) in the loop preheader  */
       vec_dest = vect_create_destination_var (scalar_dest, vectype);
@@ -2726,6 +2723,7 @@ vectorizable_load (tree stmt, block_stmt
         {
           /* Create permutation mask, if required, in loop preheader.  */
           tree builtin_decl;
+              tree params;
           params = build_tree_list (NULL_TREE, init_addr);
           vec_dest = vect_create_destination_var (scalar_dest, vectype);
           builtin_decl = targetm.vectorize.builtin_mask_for_load ();

--
Andreas Schwab, SuSE Labs, schwab@suse.de
SuSE Linux AG, Maxfeldstraße 5, 90409 Nürnberg, Germany
Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

Attachment: lnodiff.oct3
Description: Binary data


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