]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
Fix 'char' initialization, copy, check in 'libgomp.oacc-fortran/acc-memcpy.f90'
authorThomas Schwinge <tschwinge@baylibre.com>
Wed, 6 Mar 2024 22:18:08 +0000 (23:18 +0100)
committerThomas Schwinge <tschwinge@baylibre.com>
Thu, 14 Mar 2024 14:41:33 +0000 (15:41 +0100)
Our dear friend '-Wuninitialized' reported:

    [...]/libgomp.oacc-fortran/acc-memcpy.f90:18:27:

       18 |     char(j) = int (j, int8)
          |                           ^
    Warning: ā€˜j’ may be used uninitialized [-Wmaybe-uninitialized]
    [...]/libgomp.oacc-fortran/acc-memcpy.f90:14:20:

       14 |   integer(int8) :: j
          |                    ^
    note: ā€˜j’ was declared here

..., but actually there were other issues.

libgomp/
* testsuite/libgomp.oacc-fortran/acc-memcpy.f90: Fix 'char'
initialization, copy, check.

libgomp/testsuite/libgomp.oacc-fortran/acc-memcpy.f90

index 670dc50ff0717d30126316554b4e8a8adc0e5765..844d08a466161f0655c34d3c57d8b7e5ca59afa6 100644 (file)
@@ -11,15 +11,14 @@ program main
   integer(int8), allocatable :: char(:)
   type(c_ptr) :: dptr
   integer(c_intptr_t) :: i
-  integer(int8) :: j
 
   allocate(char(-128:127))
   do i = -128, 127
-    char(j) = int (j, int8)
+    char(i) = int (i, int8)
   end do
 
   dptr = acc_malloc (256_c_size_t)
-  call acc_memcpy_to_device (dptr, char, 255_c_size_t)
+  call acc_memcpy_to_device (dptr, char, 256_c_size_t)
 
   do i = 0, 255
     if (acc_is_present (transfer (transfer(char, i) + i, dptr), 1)) &
@@ -31,8 +30,7 @@ program main
   call acc_memcpy_from_device (char, dptr, 256_c_size_t)
 
   do i = -128, 127
-    char(i) = int (j, int8)
-    if (char(i) /= j) &
+    if (char(i) /= i) &
       stop 2
   end do