ira: Remove duplicate `memset' over `full_costs' from `assign_hard_reg'
Commit Message
Remove duplicate clearing of `full_costs' made in `assign_hard_reg',
which has been there since the beginning, i.e. commit 058e97ecf33a
("IRA has been merged into trunk"),
<https://gcc.gnu.org/ml/gcc-patches/2008-08/msg01932.html>.
gcc/
* ira-color.cc (assign_hard_reg): Remove duplicate `memset' over
`full_costs'.
---
Hi,
I find this fairly obvious, OK to apply?
Maciej
---
gcc/ira-color.cc | 1 -
1 file changed, 1 deletion(-)
gcc-ira-assign-hard-reg-full-costs-dup.diff
Comments
On 11/14/22 16:21, Maciej W. Rozycki wrote:
> Remove duplicate clearing of `full_costs' made in `assign_hard_reg',
> which has been there since the beginning, i.e. commit 058e97ecf33a
> ("IRA has been merged into trunk"),
> <https://gcc.gnu.org/ml/gcc-patches/2008-08/msg01932.html>.
>
> gcc/
> * ira-color.cc (assign_hard_reg): Remove duplicate `memset' over
> `full_costs'.
> ---
> Hi,
>
> I find this fairly obvious, OK to apply?
Seems obvious to me as well. OK.
jeff
On Mon, 14 Nov 2022, Jeff Law wrote:
> > gcc/
> > * ira-color.cc (assign_hard_reg): Remove duplicate `memset' over
> > `full_costs'.
> > ---
> > Hi,
> >
> > I find this fairly obvious, OK to apply?
>
> Seems obvious to me as well. OK.
Thanks, now committed.
Maciej
===================================================================
@@ -1961,7 +1961,6 @@ assign_hard_reg (ira_allocno_t a, bool r
aclass = ALLOCNO_CLASS (a);
class_size = ira_class_hard_regs_num[aclass];
best_hard_regno = -1;
- memset (full_costs, 0, sizeof (int) * class_size);
mem_cost = 0;
memset (costs, 0, sizeof (int) * class_size);
memset (full_costs, 0, sizeof (int) * class_size);