Re: RFR: 8337031: Improvements to CompilationMemoryStatistic [v2]

2024-07-30 Thread Ashutosh Mehra
On Tue, 30 Jul 2024 05:18:17 GMT, Thomas Stuefe wrote: >> Ashutosh Mehra has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Address review comments by Thomas S. >> >> Signed-off-by: Ashutosh Mehra > > src/hotspot/share/compiler/compil

Re: RFR: 8337031: Improvements to CompilationMemoryStatistic [v2]

2024-07-29 Thread Thomas Stuefe
On Mon, 29 Jul 2024 14:49:48 GMT, Ashutosh Mehra wrote: >> Some minor improvements to CompilationMemoryStatistic. More details are in >> [JDK-8337031](https://bugs.openjdk.org/browse/JDK-8337031) >> >> Testing: >> test/hotspot/jtreg/compiler/print/CompileCommandPrintMemStat.java >> >> test

Re: RFR: 8337031: Improvements to CompilationMemoryStatistic [v2]

2024-07-29 Thread Ashutosh Mehra
On Sat, 27 Jul 2024 05:44:14 GMT, Thomas Stuefe wrote: >> What do you mean by x macro? Do you have an example that shows the use of x >> macro? > > You use them already in your patch. > > E.g. > > > #define XX(name, whatever, desc) st->print_cr(" " LEGEND_KEY_FMT ": " #name > #desc > DO_AR

Re: RFR: 8337031: Improvements to CompilationMemoryStatistic [v2]

2024-07-29 Thread Ashutosh Mehra
> Some minor improvements to CompilationMemoryStatistic. More details are in > [JDK-8337031](https://bugs.openjdk.org/browse/JDK-8337031) > > Testing: > test/hotspot/jtreg/compiler/print/CompileCommandPrintMemStat.java > > test/hotspot/jtreg/serviceability/dcmd/compiler/CompilerMemoryStatist