[Bug fortran/34262] New: MVBITS does not work for arrays

[Bug fortran/34262] New: MVBITS does not work for arrays

[phl at kth dot se]

hades [TEST] cat bug-mvbits.f90 
program main
  implicit none

  integer :: a( 1 ), b( 1 )
  integer :: x, y

  a = 1
  b = 0
  x = 1
  y = 0

  call mvbits (a, 0, 1, b, 1)
  call mvbits (x, 0, 1, y, 1)

  write (*, *) 'a: ', a
  write (*, *) 'x: ', x
  write (*, *)
  write (*, *) 'b: ', b
  write (*, *) 'y: ', y
  write (*, *)

  if ( all (b == y) ) then
     write (*, *) 'GOOD:  b == y'
  else
     write (*, *) 'BAD:  b /= y'
  end if

end program main

hades [TEST] ./a.out 
 a:            1
 x:            1

 b:            0
 y:            2

 BAD:  b /= y

hades [TEST] gfortran --version
GNU Fortran (GCC) 4.2.1 (Ubuntu 4.2.1-5ubuntu4)
Copyright (C) 2007 Free Software Foundation, Inc.


It does not work for MacOS PowerPC either, same output. The version
I used ton my Mac was:

epsilon [TEST] gfortran --version      
GNU Fortran (GCC) 4.3.0 20071017 (experimental) [trunk revision 129405]
Copyright (C) 2007 Free Software Foundation, Inc.


-- 
           Summary: MVBITS does not work for arrays
           Product: gcc
           Version: 4.3.0
            Status: UNCONFIRMED
          Severity: minor
          Priority: P3
         Component: fortran
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: phl at kth dot se
 GCC build triplet: ?
  GCC host triplet: ?
GCC target triplet: ?


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34262

[Bug fortran/34262] MVBITS does not work for arrays

[burnus at gcc dot gnu dot org]

------- Comment #1 from burnus at gcc dot gnu dot org  2007-11-28 14:09 -------
For a scalars x and y:
   _gfortran_mvbits_i4 (&x, &C.860, &C.861, &y, &C.862);
for an arrays a and b:
   _gfortran_mvbits_i4 (&parm.2, &C.872, &C.873, &parm.3, &C.875);

It actually does not surprise me that this does not work if one replaces the
integer variable by a "struct array1_integer(kind=4)".


-- 

burnus at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|                            |32834
              nThis|                            |
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
           Keywords|                            |wrong-code
   Last reconfirmed|0000-00-00 00:00:00         |2007-11-28 14:09:34
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34262

[Bug fortran/34262] MVBITS does not work for arrays

[burnus at gcc dot gnu dot org]

------- Comment #2 from burnus at gcc dot gnu dot org  2007-11-28 14:20 -------
The following fixes it, but I need to check for unwanted side effects:

Index: intrinsic.c
===================================================================
--- intrinsic.c (revision 130489)
+++ intrinsic.c (working copy)
@@ -104,6 +104,7 @@ gfc_get_intrinsic_sub_symbol (const char
   gfc_symbol *sym;

   gfc_get_symbol (name, gfc_intrinsic_namespace, &sym);
+  sym->attr.elemental = 1;
   sym->attr.always_explicit = 1;
   sym->attr.subroutine = 1;
   sym->attr.flavor = FL_PROCEDURE;


-- 

burnus at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |burnus at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED
  GCC build triplet|?                           |
   GCC host triplet|?                           |
 GCC target triplet|?                           |
   Last reconfirmed|2007-11-28 14:09:34         |2007-11-28 14:20:53
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34262

[Bug fortran/34262] MVBITS does not work for arrays

[burnus at gcc dot gnu dot org]

------- Comment #3 from burnus at gcc dot gnu dot org  2007-11-29 14:57 -------
Subject: Bug 34262

Author: burnus
Date: Thu Nov 29 14:56:48 2007
New Revision: 130513

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=130513
Log:
2007-11-29  Tobias Burnus  <burnus@net-b.de>

        PR fortran/34262
        * intrinsic.c (gfc_get_intrinsic_sub_symbol): Add comment.
        (gfc_intrinsic_sub_interface): Copy elemental state if needed.
        * iresolve.c (gfc_resolve_mvbits): Mark procedure as elemental.

2007-11-29  Tobias Burnus  <burnus@net-b.de>

        PR fortran/34262
        * gfortran.dg/mvbits_3.f90: New.


Added:
    trunk/gcc/testsuite/gfortran.dg/mvbits_3.f90
Modified:
    trunk/gcc/fortran/ChangeLog
    trunk/gcc/fortran/intrinsic.c
    trunk/gcc/fortran/iresolve.c
    trunk/gcc/testsuite/ChangeLog


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34262

[Bug fortran/34262] MVBITS does not work for arrays

[burnus at gcc dot gnu dot org]

------- Comment #4 from burnus at gcc dot gnu dot org  2007-11-29 15:00 -------
Fixed on the trunk (4.3.0). I do not intent to backport the patch to 4.2.x and
4.1.x, but I can be convinced to do so.


-- 

burnus at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34262