On Fri, 17 Apr 2026 09:45:30 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). > > Stefan Karlsson has updated the pull request incrementally with one > additional commit since the last revision: > > Remove FREE_FAST Nice cleanup to use `decltype`! ------------- Marked as reviewed by dholmes (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/30786#pullrequestreview-4137464953
