[mpich-devel] warning: returning 'uint32_t *' (aka 'unsigned int *') from a function with result type 'int *' converts between pointers to integer types with different sign

[Bland, Wesley B.]

I’ll take a look. That’s my nasty code.




On 3/9/15, 8:24 PM, ""Antonio J. Peña"" <apenya at mcs.anl.gov> wrote:

>
>Thanks Jeff. This will be ticket #2245
>
>   http://trac.mpich.org/projects/mpich/ticket/2245
>
>
>On 03/09/2015 08:02 PM, Jeff Hammond wrote:
>> See below, but the propriety of this cast can be verified in source
>> without a compiler :-)
>>
>> Jeff
>>
>> ## Local Branches:
>>
>> * master
>>
>> ## Most Recent Commit:
>>
>> commit 957ceec598103d8d643c9345512ae8ad4e88e993
>> Author: Ken Raffenetti <raffenet at mcs.anl.gov>
>>
>>      testsuite: remove leading spaces in testlist files
>>
>> jrhammon-mac01:build jrhammon$ head config.log
>> This file contains any messages produced by compilers while
>> running configure, to aid debugging if configure makes a mistake.
>>
>> It was created by MPICH configure 3.2b1, which was
>> generated by GNU Autoconf 2.69.  Invocation command line was
>>
>>    $ ../configure CC=icc CXX=icpc FC=ifort F77=ifort --enable-fortran
>> --enable-threads=runtime --enable-g=dbg --with-pm=hydra
>> --prefix=/opt/mpich/dev/intel/default --enable-wrapper-rpath
>> --enable-static --enable-shared
>>
>> jrhammon-mac01:build jrhammon$ icc --version
>> icc (ICC) 15.0.1 20141022
>> Copyright (C) 1985-2014 Intel Corporation.  All rights reserved.
>>
>>
>>
>> On Mon, Mar 9, 2015 at 5:58 PM, "Antonio J. Peña" <apenya at mcs.anl.gov> wrote:
>>> Hi Jeff,
>>>
>>> Please also report compiler, compiler version, and MPICH version.
>>>
>>> Best,
>>>    Toni
>>>
>>>
>>>
>>> On 03/09/2015 07:53 PM, Jeff Hammond wrote:
>>>> I know some of you like warning-clean builds...
>>>>
>>>> ../src/mpid/ch3/src/mpid_comm_get_all_failed_procs.c:20:14: warning:
>>>> assigning to 'uint32_t *' (aka 'unsigned int *') from 'int *'
>>>>         converts between pointers to integer types with different sign
>>>> [-Wpointer-sign]
>>>>       bitarray = (int *) MPIU_Malloc(sizeof(int) * bitarray_size);
>>>>                ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>>>> ../src/mpid/ch3/src/mpid_comm_get_all_failed_procs.c:24:16: warning:
>>>> returning 'uint32_t *' (aka 'unsigned int *') from a function
>>>>         with result type 'int *' converts between pointers to integer
>>>> types with different sign [-Wpointer-sign]
>>>>           return bitarray;
>>>>                  ^~~~~~~~
>>>> ../src/mpid/ch3/src/mpid_comm_get_all_failed_procs.c:46:12: warning:
>>>> returning 'uint32_t *' (aka 'unsigned int *') from a function
>>>>         with result type 'int *' converts between pointers to integer
>>>> types with different sign [-Wpointer-sign]
>>>>       return bitarray;
>>>>              ^~~~~~~~
>>>> ../src/mpid/ch3/src/mpid_comm_get_all_failed_procs.c:107:14: warning:
>>>> assigning to 'uint32_t *' (aka 'unsigned int *') from
>>>>         'int *' converts between pointers to integer types with
>>>> different sign [-Wpointer-sign]
>>>>       bitarray = group_to_bitarray(local_fail, comm_ptr);
>>>>                ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>>>> 4 warnings generated.
>>>>
>>>> Best,
>>>>
>>>> Jeff
>>>>
>>>
>>> --
>>> Antonio J. Peña
>>> Postdoctoral Appointee
>>> Mathematics and Computer Science Division
>>> Argonne National Laboratory
>>> 9700 South Cass Avenue, Bldg. 240, Of. 3148
>>> Argonne, IL 60439-4847
>>> apenya at mcs.anl.gov
>>> www.mcs.anl.gov/~apenya
>>>
>>> _______________________________________________
>>> To manage subscription options or unsubscribe:
>>> https://lists.mpich.org/mailman/listinfo/devel
>>
>>
>
>
>-- 
>Antonio J. Peña
>Postdoctoral Appointee
>Mathematics and Computer Science Division
>Argonne National Laboratory
>9700 South Cass Avenue, Bldg. 240, Of. 3148
>Argonne, IL 60439-4847
>apenya at mcs.anl.gov
>www.mcs.anl.gov/~apenya
>
>_______________________________________________
>To manage subscription options or unsubscribe:
>https://lists.mpich.org/mailman/listinfo/devel