[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-hotspot-runtime-dev
Subject:    Re: RFR: 8281455: Change JVM options with small ranges from 64 to 32 bits, for gc_globals.hpp [v4]
From:       Afshin Zafari <azafari () openjdk ! org>
Date:       2023-06-27 15:58:18
Message-ID: qudRozZoxI3R9z9UFqYqjXETxdfveP2Lxp4Oocla0EI=.5a703b95-eac2-4e39-8b61-43184f053d69 () github ! com
[Download RAW message or body]

On Mon, 26 Jun 2023 11:46:21 GMT, Afshin Zafari <azafari@openjdk.org> wrote:

> > The `uintx/intx` options whose ranges are small enough are changed to `uint/int`, \
> > otherwise gcc complains  when `-Wconversion` is used in build.
> > Their uses in printf formats are changed accordingly.
> > 
> > ### Tests
> > Locally hotspot:tier1 tested on linux-x64
> > mach5 tiers 1-4 on Linux and Windows 64bits platforms passed.
> 
> Afshin Zafari has updated the pull request incrementally with one additional commit \
> since the last revision: 
> review comments applied.

Thank you Coleen, Kim, Thomas and Amit for your comments and reviewing this PR.

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

PR Comment: https://git.openjdk.org/jdk/pull/14259#issuecomment-1609796995


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic