Opened 3 months ago

Last modified 3 months ago

#15651 new task

Check if some auto apply code is dead and remove if appropriate.

Reported by: AndreasK Owned by:
Priority: normal Milestone: 8.8.1
Component: Runtime System Version: 8.4.3
Keywords: Newcomer Cc: osa1, sgraf
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: None/Unknown Test Case:
Blocked By: Blocking:
Related Tickets: Differential Rev(s):
Wiki Page:

Description

There is a whole family of stg_ap_stk_* and stg_stk_save_* functions generated in AutoApply.cmm which as far as I can tell is not used anywhere in the compiler and can likely be removed.

In particular this would involve:

  • Stop the code in question from being generated. (utils/genapply)
  • Make sure that doesn't break things. (Validate/Run the testsuite).
  • Grep for stk in the compiler to make sure we don't build calls to these in rare circumstances with string concatenation for extra safety.
  • Document the change in patch notes just in case.

Change History (2)

comment:1 Changed 3 months ago by osa1

Cc: osa1 added

comment:2 Changed 3 months ago by sgraf

Cc: sgraf added
Milestone: 8.6.18.8.1
Note: See TracTickets for help on using tickets.