On Mon, 13 Jan 2025 15:49:15 GMT, Coleen Phillimore <cole...@openjdk.org> wrote:

>> There are a lot of format modifiers that are noisy and unnecessary in the 
>> code.  This change removes the INTX variants.  It's not that disruptive even 
>> for backporting because %z modifier has been available for a long time so 
>> should backport fine.  This was mostly done with a sed script plus some hand 
>> fixups.
>> 
>> Testing mach5 and other platform cross compilations in progress.  Opening 
>> this for GHA testing.
>
> Coleen Phillimore has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Add Oracle copyright to shenandoah files for this change.

We have belatedly discovered that `0x%zx` and `%#zx` behave differently in 
their handling of zero. The former prints `0x0` while the latter just prints 
`0`. This has broken the compiler replay tests as the parsing of 0 no longer 
works.

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

PR Comment: https://git.openjdk.org/jdk/pull/22916#issuecomment-2588550581

Reply via email to