RFR: 8314265: Fix -Wconversion warnings in miscellaneous runtime code [v7]

Coleen Phillimore coleenp at openjdk.org
Thu Aug 17 12:37:05 UTC 2023


On Thu, 17 Aug 2023 07:52:07 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   put sizes back to uint32_t.
>
> src/hotspot/share/utilities/elfFuncDescTable.cpp line 49:
> 
>> 47: }
>> 48: 
>> 49: address ElfFuncDescTable::lookup(Elf64_Addr index) {
> 
> I think this could be `Elf_Addr` to be platform-specific. `Elf_Word` is either `Elf32_Word` or `Elf64_Word`, depending on the platform. But both are `typedefs` for `uint32_t` for some reason. That's why it must have complained here.

Ok, thanks, I'll change that.  I picked the type that the compiler told me, but this would be better on 32 bit platforms.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/15233#discussion_r1297137694


More information about the hotspot-dev mailing list