]> gcc.gnu.org Git - gcc.git/blobdiff - libgfortran/intrinsics/pack_generic.c
re PR libfortran/22423 (Warnings when building libgfortran)
[gcc.git] / libgfortran / intrinsics / pack_generic.c
index da0bc57982710142cb986cac5d200483215f6f5b..61b41e53e65feae0fc84f938310290812163d521 100644 (file)
@@ -1,38 +1,46 @@
 /* Generic implementation of the PACK intrinsic
-   Copyright (C) 2002, 2004 Free Software Foundation, Inc.
+   Copyright (C) 2002, 2004, 2005, 2006, 2007 Free Software Foundation, Inc.
    Contributed by Paul Brook <paul@nowt.org>
 
-This file is part of the GNU Fortran 95 runtime library (libgfor).
+This file is part of the GNU Fortran 95 runtime library (libgfortran).
 
-Libgfor is free software; you can redistribute it and/or
-modify it under the terms of the GNU Lesser General Public
+Libgfortran is free software; you can redistribute it and/or
+modify it under the terms of the GNU General Public
 License as published by the Free Software Foundation; either
-version 2.1 of the License, or (at your option) any later version.
-
-Ligbfor is distributed in the hope that it will be useful,
+version 2 of the License, or (at your option) any later version.
+
+In addition to the permissions in the GNU General Public License, the
+Free Software Foundation gives you unlimited permission to link the
+compiled version of this file into combinations with other programs,
+and to distribute those combinations without any restriction coming
+from the use of this file.  (The General Public License restrictions
+do apply in other respects; for example, they cover modification of
+the file, and distribution when not linked into a combine
+executable.)
+
+Ligbfortran is distributed in the hope that it will be useful,
 but WITHOUT ANY WARRANTY; without even the implied warranty of
 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-GNU Lesser General Public License for more details.
+GNU General Public License for more details.
 
-You should have received a copy of the GNU Lesser General Public
-License along with libgfor; see the file COPYING.LIB.  If not,
-write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
-Boston, MA 02111-1307, USA.  */
+You should have received a copy of the GNU General Public
+License along with libgfortran; see the file COPYING.  If not,
+write to the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
+Boston, MA 02110-1301, USA.  */
 
-#include "config.h"
+#include "libgfortran.h"
 #include <stdlib.h>
 #include <assert.h>
 #include <string.h>
-#include "libgfortran.h"
 
 /* PACK is specified as follows:
 
    13.14.80 PACK (ARRAY, MASK, [VECTOR])
-   
+
    Description: Pack an array into an array of rank one under the
    control of a mask.
 
-   Class: Transformational fucntion.
+   Class: Transformational function.
 
    Arguments:
       ARRAY   may be of any type. It shall not be scalar.
@@ -40,7 +48,7 @@ Boston, MA 02111-1307, USA.  */
       VECTOR  (optional) shall be of the same type and type parameters
               as ARRAY. VECTOR shall have at least as many elements as
               there are true elements in MASK. If MASK is a scalar
-              with the value true, VECTOR shall have at least as many 
+              with the value true, VECTOR shall have at least as many
               elements as there are in ARRAY.
 
    Result Characteristics: The result is an array of rank one with the
@@ -60,14 +68,15 @@ Boston, MA 02111-1307, USA.  */
    | 9 0 0 | may be "gathered" by the function PACK. The result of
    | 0 0 7 |
    PACK (M, MASK = M.NE.0) is [9,7] and the result of PACK (M, M.NE.0,
-   VECTOR = (/ 2,4,6,8,10,12 /)) is [9,7,6,8,10,12].  
+   VECTOR = (/ 2,4,6,8,10,12 /)) is [9,7,6,8,10,12].
 
 There are two variants of the PACK intrinsic: one, where MASK is
 array valued, and the other one where MASK is scalar.  */
 
-void
-__pack (gfc_array_char * ret, const gfc_array_char * array,
-       const gfc_array_l4 * mask, const gfc_array_char * vector)
+static void
+pack_internal (gfc_array_char *ret, const gfc_array_char *array,
+              const gfc_array_l1 *mask, const gfc_array_char *vector,
+              index_type size)
 {
   /* r.* indicates the return array.  */
   index_type rstride0;
@@ -79,57 +88,69 @@ __pack (gfc_array_char * ret, const gfc_array_char * array,
   /* m.* indicates the mask array.  */
   index_type mstride[GFC_MAX_DIMENSIONS];
   index_type mstride0;
-  const GFC_LOGICAL_4 *mptr;
+  const GFC_LOGICAL_1 *mptr;
 
   index_type count[GFC_MAX_DIMENSIONS];
   index_type extent[GFC_MAX_DIMENSIONS];
+  int zero_sized;
   index_type n;
   index_type dim;
-  index_type size;
   index_type nelem;
+  index_type total;
+  int mask_kind;
 
-  size = GFC_DESCRIPTOR_SIZE (array);
   dim = GFC_DESCRIPTOR_RANK (array);
+
+  sptr = array->data;
+  mptr = mask->data;
+
+  /* Use the same loop for all logical types, by using GFC_LOGICAL_1
+     and using shifting to address size and endian issues.  */
+
+  mask_kind = GFC_DESCRIPTOR_SIZE (mask);
+
+  if (mask_kind == 1 || mask_kind == 2 || mask_kind == 4 || mask_kind == 8
+#ifdef HAVE_GFC_LOGICAL_16
+      || mask_kind == 16
+#endif
+      )
+    {
+      /*  Don't convert a NULL pointer as we use test for NULL below.  */
+      if (mptr)
+       mptr = GFOR_POINTER_TO_L1 (mptr, mask_kind);
+    }
+  else
+    runtime_error ("Funny sized logical array");
+
+  zero_sized = 0;
   for (n = 0; n < dim; n++)
     {
       count[n] = 0;
       extent[n] = array->dim[n].ubound + 1 - array->dim[n].lbound;
+      if (extent[n] <= 0)
+       zero_sized = 1;
       sstride[n] = array->dim[n].stride * size;
-      mstride[n] = mask->dim[n].stride;
+      mstride[n] = mask->dim[n].stride * mask_kind;
     }
   if (sstride[0] == 0)
     sstride[0] = size;
   if (mstride[0] == 0)
-    mstride[0] = 1;
-
-  sptr = array->data;
-  mptr = mask->data;
-
-  /* Use the same loop for both logical types. */
-  if (GFC_DESCRIPTOR_SIZE (mask) != 4)
-    {
-      if (GFC_DESCRIPTOR_SIZE (mask) != 8)
-        runtime_error ("Funny sized logical array");
-      for (n = 0; n < dim; n++)
-        mstride[n] <<= 1;
-      mstride0 <<= 1;
-      mptr = GFOR_POINTER_L8_TO_L4 (mptr);
-    }
+    mstride[0] = mask_kind;
 
-  if (ret->data == NULL)
+  if (ret->data == NULL || compile_options.bounds_check)
     {
-      /* Allocate the memory for the result.  */
-      int total;
+      /* Count the elements, either for allocating memory or
+        for bounds checking.  */
 
-      if (vector != NULL) 
-       { 
+      if (vector != NULL)
+       {
          /* The return array will have as many
-            elements as there are in VECTOR.  */ 
-         total = vector->dim[0].ubound + 1 - vector->dim[0].lbound; 
-       } 
-      else 
-       { 
-         /* We have to count the true elements in MASK.  */ 
+            elements as there are in VECTOR.  */
+         total = vector->dim[0].ubound + 1 - vector->dim[0].lbound;
+       }
+      else
+       {
+         /* We have to count the true elements in MASK.  */
 
          /* TODO: We could speed up pack easily in the case of only
             few .TRUE. entries in MASK, by keeping track of where we
@@ -139,14 +160,16 @@ __pack (gfc_array_char * ret, const gfc_array_char * array,
             only have to traverse the list, and copy those elements
             into the result array. In the case of datatypes which fit
             in one of the integer types we could also cache the
-            value instead of a pointer to it. 
+            value instead of a pointer to it.
             This approach might be bad from the point of view of
             cache behavior in the case where our cache is not big
             enough to hold all elements that have to be copied.  */
 
-         const GFC_LOGICAL_4 *m = mptr;
+         const GFC_LOGICAL_1 *m = mptr;
 
          total = 0;
+         if (zero_sized)
+           m = NULL;
 
          while (m)
            {
@@ -164,7 +187,7 @@ __pack (gfc_array_char * ret, const gfc_array_char * array,
                     and increment the next dimension.  */
                  count[n] = 0;
                  /* We could precalculate this product, but this is a
-                    less frequently used path so proabably not worth
+                    less frequently used path so probably not worth
                     it.  */
                  m -= mstride[n] * extent[n];
                  n++;
@@ -177,23 +200,40 @@ __pack (gfc_array_char * ret, const gfc_array_char * array,
                  else
                    {
                      count[n]++;
-                     mptr += mstride[n];
+                     m += mstride[n];
                    }
                }
            }
        }
-      
-      /* Setup the array descriptor.  */
-      ret->dim[0].lbound = 0;
-      ret->dim[0].ubound = total - 1;
-      ret->dim[0].stride = 1;
 
-      ret->data = internal_malloc (size * total);
-      ret->base = 0;
+      if (ret->data == NULL)
+       {
+         /* Setup the array descriptor.  */
+         ret->dim[0].lbound = 0;
+         ret->dim[0].ubound = total - 1;
+         ret->dim[0].stride = 1;
 
-      if (total == 0)
-       /* In this case, nothing remains to be done.  */
-       return;
+         ret->offset = 0;
+         if (total == 0)
+           {
+             /* In this case, nothing remains to be done.  */
+             ret->data = internal_malloc_size (1);
+             return;
+           }
+         else
+           ret->data = internal_malloc_size (size * total);
+       }
+      else 
+       {
+         /* We come here because of range checking.  */
+         index_type ret_extent;
+
+         ret_extent = ret->dim[0].ubound + 1 - ret->dim[0].lbound;
+         if (total != ret_extent)
+           runtime_error ("Incorrect extent in return value of PACK intrinsic;"
+                          " is %ld, should be %ld", (long int) total,
+                          (long int) ret_extent);
+       }
     }
 
   rstride0 = ret->dim[0].stride * size;
@@ -203,7 +243,7 @@ __pack (gfc_array_char * ret, const gfc_array_char * array,
   mstride0 = mstride[0];
   rptr = ret->data;
 
-  while (sptr)
+  while (sptr && mptr)
     {
       /* Test this element.  */
       if (*mptr)
@@ -223,7 +263,7 @@ __pack (gfc_array_char * ret, const gfc_array_char * array,
              the next dimension.  */
           count[n] = 0;
           /* We could precalculate these products, but this is a less
-             frequently used path so proabably not worth it.  */
+             frequently used path so probably not worth it.  */
           sptr -= sstride[n] * extent[n];
           mptr -= mstride[n] * extent[n];
           n++;
@@ -265,9 +305,36 @@ __pack (gfc_array_char * ret, const gfc_array_char * array,
     }
 }
 
+extern void pack (gfc_array_char *, const gfc_array_char *,
+                 const gfc_array_l1 *, const gfc_array_char *);
+export_proto(pack);
+
+void
+pack (gfc_array_char *ret, const gfc_array_char *array,
+      const gfc_array_l1 *mask, const gfc_array_char *vector)
+{
+  pack_internal (ret, array, mask, vector, GFC_DESCRIPTOR_SIZE (array));
+}
+
+extern void pack_char (gfc_array_char *, GFC_INTEGER_4, const gfc_array_char *,
+                      const gfc_array_l1 *, const gfc_array_char *,
+                      GFC_INTEGER_4, GFC_INTEGER_4);
+export_proto(pack_char);
+
 void
-__pack_s (gfc_array_char * ret, const gfc_array_char * array,
-         const GFC_LOGICAL_4 * mask, const gfc_array_char * vector)
+pack_char (gfc_array_char *ret,
+          GFC_INTEGER_4 ret_length __attribute__((unused)),
+          const gfc_array_char *array, const gfc_array_l1 *mask,
+          const gfc_array_char *vector, GFC_INTEGER_4 array_length,
+          GFC_INTEGER_4 vector_length __attribute__((unused)))
+{
+  pack_internal (ret, array, mask, vector, array_length);
+}
+
+static void
+pack_s_internal (gfc_array_char *ret, const gfc_array_char *array,
+                const GFC_LOGICAL_4 *mask, const gfc_array_char *vector,
+                index_type size)
 {
   /* r.* indicates the return array.  */
   index_type rstride0;
@@ -281,16 +348,17 @@ __pack_s (gfc_array_char * ret, const gfc_array_char * array,
   index_type extent[GFC_MAX_DIMENSIONS];
   index_type n;
   index_type dim;
-  index_type size;
+  index_type ssize;
   index_type nelem;
 
-  size = GFC_DESCRIPTOR_SIZE (array);
   dim = GFC_DESCRIPTOR_RANK (array);
+  ssize = 1;
   for (n = 0; n < dim; n++)
     {
       count[n] = 0;
       extent[n] = array->dim[n].ubound + 1 - array->dim[n].lbound;
       sstride[n] = array->dim[n].stride * size;
+      ssize *= extent[n];
     }
   if (sstride[0] == 0)
     sstride[0] = size;
@@ -320,25 +388,23 @@ __pack_s (gfc_array_char * ret, const gfc_array_char * array,
                total *= extent[n];
            }
          else
-           {
-             /* The result array will be empty.  */
-             ret->dim[0].lbound = 0;
-             ret->dim[0].ubound = -1;
-             ret->dim[0].stride = 1;
-             ret->data = internal_malloc (0);
-             ret->base = 0;
-             
-             return;
-           }
+           /* The result array will be empty.  */
+           total = 0;
        }
 
       /* Setup the array descriptor.  */
       ret->dim[0].lbound = 0;
       ret->dim[0].ubound = total - 1;
       ret->dim[0].stride = 1;
+      ret->offset = 0;
 
-      ret->data = internal_malloc (size * total);
-      ret->base = 0;
+      if (total == 0)
+       {
+         ret->data = internal_malloc_size (1);
+         return;
+       }
+      else
+       ret->data = internal_malloc_size (size * total);
     }
 
   rstride0 = ret->dim[0].stride * size;
@@ -346,13 +412,13 @@ __pack_s (gfc_array_char * ret, const gfc_array_char * array,
     rstride0 = size;
   rptr = ret->data;
 
-  /* The remaining possibilities are now: 
+  /* The remaining possibilities are now:
        If MASK is .TRUE., we have to copy the source array into the
      result array. We then have to fill it up with elements from VECTOR.
        If MASK is .FALSE., we have to copy VECTOR into the result
      array. If VECTOR were not present we would have already returned.  */
 
-  if (*mask)
+  if (*mask && ssize != 0)
     {
       while (sptr)
        {
@@ -370,7 +436,7 @@ __pack_s (gfc_array_char * ret, const gfc_array_char * array,
                 increment the next dimension.  */
              count[n] = 0;
              /* We could precalculate these products, but this is a
-                less frequently used path so proabably not worth it.  */
+                less frequently used path so probably not worth it.  */
              sptr -= sstride[n] * extent[n];
              n++;
              if (n >= dim)
@@ -387,7 +453,7 @@ __pack_s (gfc_array_char * ret, const gfc_array_char * array,
            }
        }
     }
-  
+
   /* Add any remaining elements from VECTOR.  */
   if (vector)
     {
@@ -410,3 +476,30 @@ __pack_s (gfc_array_char * ret, const gfc_array_char * array,
         }
     }
 }
+
+extern void pack_s (gfc_array_char *ret, const gfc_array_char *array,
+                   const GFC_LOGICAL_4 *, const gfc_array_char *);
+export_proto(pack_s);
+
+void
+pack_s (gfc_array_char *ret, const gfc_array_char *array,
+       const GFC_LOGICAL_4 *mask, const gfc_array_char *vector)
+{
+  pack_s_internal (ret, array, mask, vector, GFC_DESCRIPTOR_SIZE (array));
+}
+
+extern void pack_s_char (gfc_array_char *ret, GFC_INTEGER_4,
+                        const gfc_array_char *array, const GFC_LOGICAL_4 *,
+                        const gfc_array_char *, GFC_INTEGER_4,
+                        GFC_INTEGER_4);
+export_proto(pack_s_char);
+
+void
+pack_s_char (gfc_array_char *ret,
+            GFC_INTEGER_4 ret_length __attribute__((unused)),
+            const gfc_array_char *array, const GFC_LOGICAL_4 *mask,
+            const gfc_array_char *vector, GFC_INTEGER_4 array_length,
+            GFC_INTEGER_4 vector_length __attribute__((unused)))
+{
+  pack_s_internal (ret, array, mask, vector, array_length);
+}
This page took 0.044847 seconds and 5 git commands to generate.