Dear scheduler maintainers,
Gentle ping. This is a trivial cleanup.
On Wed, 22 Nov 2023 at 15:14, Maxim Kuvyrkov <maxim.kuvyr...@linaro.org>
wrote:
> gcc/ChangeLog:
>
> * sched-deps.cc (init_deps, init_deps_reg_last): Simplify.
> (free_deps): Remove useless code.
> ---
> gcc/sched-deps.cc | 13 ++++---------
> 1 file changed, 4 insertions(+), 9 deletions(-)
>
> diff --git a/gcc/sched-deps.cc b/gcc/sched-deps.cc
> index 2a87158ba4b..e0d3c97d935 100644
> --- a/gcc/sched-deps.cc
> +++ b/gcc/sched-deps.cc
> @@ -3927,10 +3927,9 @@ init_deps (class deps_desc *deps, bool
> lazy_reg_last)
> int max_reg = (reload_completed ? FIRST_PSEUDO_REGISTER : max_reg_num
> ());
>
> deps->max_reg = max_reg;
> - if (lazy_reg_last)
> - deps->reg_last = NULL;
> - else
> - deps->reg_last = XCNEWVEC (struct deps_reg, max_reg);
> + deps->reg_last = NULL;
> + if (!lazy_reg_last)
> + init_deps_reg_last (deps);
> INIT_REG_SET (&deps->reg_last_in_use);
>
> deps->pending_read_insns = 0;
> @@ -3961,9 +3960,7 @@ init_deps (class deps_desc *deps, bool lazy_reg_last)
> void
> init_deps_reg_last (class deps_desc *deps)
> {
> - gcc_assert (deps && deps->max_reg > 0);
> - gcc_assert (deps->reg_last == NULL);
> -
> + gcc_assert (deps && deps->max_reg > 0 && deps->reg_last == NULL);
> deps->reg_last = XCNEWVEC (struct deps_reg, deps->max_reg);
> }
>
> @@ -4013,8 +4010,6 @@ free_deps (class deps_desc *deps)
> it at all. */
> free (deps->reg_last);
> deps->reg_last = NULL;
> -
> - deps = NULL;
> }
>
> /* Remove INSN from dependence contexts DEPS. */
> --
> 2.34.1
>
>
--
Maxim Kuvyrkov
www.linaro.org
