Menu ▾ ▴

#872 stm8 stdcbench code size and score regression

None
closed-fixed
None
5
2023-07-28
2023-03-01
No

For stm8, stdcbench score and code size regressed substantially comparing [r13800] vs [r13850].

Related

Commit: [r13800]
Commit: [r13850]

Discussion

  • Philipp Klaus Krause

    It's the strtoul fix in [r13811]. Nothing to be done about it now. The way forward here is a more efficient ckd_mul.

     

    Related

    Commit: [r13811]

  • Philipp Klaus Krause

    Ticket moved from /p/sdcc/bugs/3562/

    Can't be converted:

    • _category: STM8
     
  • Philipp Klaus Krause

    In the next branch, we now have an optimization that improved ckd_mul performance, but there is another problem on ckd_add contributing to the regression: [bugs:#3602].

     

    Related

    Feature Requests: #880


    Last edit: Philipp Klaus Krause 2023-06-11
  • Philipp Klaus Krause

    In the next branch, the implementation of chk_add has been changed so that all calls get inlined; code size should be back down to normal levels now.
    But there is still a bit more potential for further optimization by using cheaper types. Either manually, or automatically as an optimization based on information from generalized constant propagation (which is currently being developed in the genconstprop branch).

     
  • Philipp Klaus Krause

    • status: open --> closed-fixed
    • assigned_to: Philipp Klaus Krause
    • Group: -->
     
  • Philipp Klaus Krause

    Fixed by merging the next and genconstprop branches to trunk a while ago.

     

Log in to post a comment.