On 7/2/2026 6:59 PM, Léo Hardt wrote:
Good evening,
The two declarations mentioned below are not used.
This patch removes them from the code.
* current_succs was introduced in e855c69, and
it is not used since then. Perhaps a previous
working version of the code used it and was
refactored away. One may note that there is
only one commit for 'git log -S current_succs',
and that it was featured on Jan's "Unused Vars"
e-mail from the year after it was added:
https://gcc.gnu.org/legacy-ml/gcc/2010-05/msg00688.html
If I might speculate, the comment suggests it was
supposed to hold the value of compute_succs_info,
but in the productive code, the value is simply
returned.
* ggc_force_collect is an old variable, which
was made into a parameter of ggc_collect in
0edf2e81. An 'extern' declaration for it was
later added in 1eee94d, when merging the gm2
front-end into this repository.
Bootstrapped successfully on my machine.
gcc/m2/ChangeLog:
* plugin/m2rte.cc: Removed unused extern bool
ggc_force_collect.
gcc/ChangeLog:
* sel-sched-ir.cc (struct succs_info): Removed
unused variable current_succs.
Signed-off-by: Léo Hardt <[email protected]>
THanks. I took care of the bootstrap & regression test requirement on
x86 & riscv64 (which not surprisingly passed) and have pushed this to
the trunk after trivial ChangeLog formatting fixes.
jeff
