Re: RFR: 8334328: Reduce object allocation for FloatToDecimal and DoubleToDecimal [v20]

2024-06-26 Thread Chen Liang
On Wed, 26 Jun 2024 14:55:44 GMT, Shaojin Wen wrote: >> The current versions of FloatToDecimal and DoubleToDecimal allocate >> additional objects. Reducing these allocations can improve the performance >> of Float/Double.toString and AbstractStringBuilder's append(float/double). >> >> This pat

Re: RFR: 8334328: Reduce object allocation for FloatToDecimal and DoubleToDecimal [v20]

2024-06-26 Thread Shaojin Wen
On Wed, 26 Jun 2024 17:42:13 GMT, Chen Liang wrote: >> Shaojin Wen has updated the pull request incrementally with one additional >> commit since the last revision: >> >> 1. revert code change. >> 2. comment remove space > > src/java.base/share/classes/jdk/internal/math/DoubleToDecimal.java

Re: RFR: 8334328: Reduce object allocation for FloatToDecimal and DoubleToDecimal [v20]

2024-06-26 Thread Claes Redestad
On Wed, 26 Jun 2024 14:55:44 GMT, Shaojin Wen wrote: >> The current versions of FloatToDecimal and DoubleToDecimal allocate >> additional objects. Reducing these allocations can improve the performance >> of Float/Double.toString and AbstractStringBuilder's append(float/double). >> >> This pat

Re: RFR: 8334328: Reduce object allocation for FloatToDecimal and DoubleToDecimal [v20]

2024-06-26 Thread Chen Liang
On Wed, 26 Jun 2024 14:55:44 GMT, Shaojin Wen wrote: >> The current versions of FloatToDecimal and DoubleToDecimal allocate >> additional objects. Reducing these allocations can improve the performance >> of Float/Double.toString and AbstractStringBuilder's append(float/double). >> >> This pat

Re: RFR: 8334328: Reduce object allocation for FloatToDecimal and DoubleToDecimal [v20]

2024-06-26 Thread Raffaello Giulietti
On Wed, 26 Jun 2024 14:55:44 GMT, Shaojin Wen wrote: >> The current versions of FloatToDecimal and DoubleToDecimal allocate >> additional objects. Reducing these allocations can improve the performance >> of Float/Double.toString and AbstractStringBuilder's append(float/double). >> >> This pat

Re: RFR: 8334328: Reduce object allocation for FloatToDecimal and DoubleToDecimal [v20]

2024-06-26 Thread Shaojin Wen
> The current versions of FloatToDecimal and DoubleToDecimal allocate > additional objects. Reducing these allocations can improve the performance of > Float/Double.toString and AbstractStringBuilder's append(float/double). > > This patch is just a code refactoring to reduce object allocation, b