[Mesa-dev] [PATCH] util: fix warning/error on 32bit build

Michel Dänzer michel at daenzer.net
Tue Jul 25 06:08:04 UTC 2017


On 25/07/17 03:02 PM, Michel Dänzer wrote:
> On 25/07/17 02:42 PM, Tapani Pälli wrote:
>> Add uintptr_t cast to fix 'cast to pointer from integer of different size'
>> warning on 32bit build (build error on Android M).
>>
>> Signed-off-by: Tapani Pälli <tapani.palli at intel.com>
>> ---
>>  src/util/hash_table.c | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/src/util/hash_table.c b/src/util/hash_table.c
>> index a9d442d..1bda214 100644
>> --- a/src/util/hash_table.c
>> +++ b/src/util/hash_table.c
>> @@ -588,7 +588,7 @@ _mesa_hash_table_u64_insert(struct hash_table_u64 *ht, uint64_t key,
>>     }
>>  
>>     if (sizeof(void *) == 8) {
>> -      _mesa_hash_table_insert(ht->table, (void *)key, data);
>> +      _mesa_hash_table_insert(ht->table, (void *)(uintptr_t)key, data);
>>     } else {
>>        struct hash_key_u64 *_key = CALLOC_STRUCT(hash_key_u64);
>>  
>> @@ -604,7 +604,7 @@ static struct hash_entry *
>>  hash_table_u64_search(struct hash_table_u64 *ht, uint64_t key)
>>  {
>>     if (sizeof(void *) == 8) {
>> -      return _mesa_hash_table_search(ht->table, (void *)key);
>> +      return _mesa_hash_table_search(ht->table, (void *)(uintptr_t)key);
>>     } else {
>>        struct hash_key_u64 _key = { .value = key };
>>        return _mesa_hash_table_search(ht->table, &_key);
>>
> 
> This means that the upper 32 bits of the key are ignored on 32-bit
> platforms, right? Is that okay?

Never mind, I missed that the changed code is guarded by if (sizeof(void
*) == 8).

Reviewed-by: Michel Dänzer <michel.daenzer at amd.com>


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer


More information about the mesa-dev mailing list