On Fri, 17 Apr 2026 07:35:02 GMT, Stefan Karlsson <[email protected]> wrote:
> In JDK-8381934 "Wrong type passed to FREE_C_HEAP_ARRAY deallocating > G1CardSetMemoryManager" we saw how the wrong type was being passed to the > FREE_C_HEAP_ARRAY macro. That bug is benign, because the type parameter is > not used. I see that we have similar issues in some of our gtests. I propose > that we get rid of the type parameter to remove this confusion and lower the > line noise on these lines. > > I also propose that we get rid of it from the REALLOC_ macros. These macros > however are using the type parameter, but we can infer the type from the > other parameters, so I propose that we do that instead. > > Tested with tier1-3 > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). This pull request has now been integrated. Changeset: abd3d462 Author: Stefan Karlsson <[email protected]> URL: https://git.openjdk.org/jdk/commit/abd3d4620423c322d18e7d8a4f3476de97402054 Stats: 443 lines in 184 files changed: 4 ins; 8 del; 431 mod 8382401: Remove return type parameters from FREE_ and REALLOC_ macros Reviewed-by: tschatzl, dholmes ------------- PR: https://git.openjdk.org/jdk/pull/30786
