Menu

#2987 gcc-torture-execute-strlen-4.c fails

pending-fixed
None
other
5
2023-12-29
2020-03-22
No

Now that gcc-torture-execute-strlen-4.c compiles, the test still fails when enabled.
We need to check if that is okay (maybe it just relies on GCC implementation-defined behaviour with SDCC making different choices) or a real SDCC bug.

Related

Bugs: #2987

Discussion

  • Philipp Klaus Krause

    Looking further into this, it looks like real SDCC bugs.

    • For all ports it fails in test_ptr_array.
    • For stm8, it failed in test_array_ptr due to a bug in handling signed char literals; I fixed it in [r11559].
    • For hc08 and s08 it fails for some other reason in test_array_ptr.
     

    Last edit: Philipp Klaus Krause 2023-01-26
  • Philipp Klaus Krause

    I can still reproduce this issue on my Debian GNU/Linux testing system as of [r13824].

     

    Related

    Commit: [r13824]

  • Gabriele Gorla

    Gabriele Gorla - 2023-12-24

    I root caused the bug in incorrect handling of a signed byte in the hc08 codegen that was also copied to the mos6502 ports.

     
  • Gabriele Gorla

    Gabriele Gorla - 2023-12-24
    • assigned_to: Gabriele Gorla
     
  • Gabriele Gorla

    Gabriele Gorla - 2023-12-29

    [r14561] fixes the issue for both mos6502 and hc08

     

    Related

    Commit: [r14561]


    Last edit: Gabriele Gorla 2023-12-29
  • Gabriele Gorla

    Gabriele Gorla - 2023-12-29
    • status: open --> pending-fixed
     
  • Maarten Brock

    Maarten Brock - 2023-12-29

    If you replace the tabs with spaces as it should be it would look well in every view / editor.
    Have a look at the commit link [r14561] to see how it currently fails.

     

    Related

    Commit: [r14561]

    • Gabriele Gorla

      Gabriele Gorla - 2023-12-30

      You're right.
      I need to be more careful with the editor settings.
      GG

      On Friday, December 29, 2023 at 07:17:52 AM PST, Maarten Brock maartenbrock@users.sourceforge.net wrote:

      If you replace the tabs with spaces as it should be it would look well in every view / editor.
      Have a look at the commit link [r14561] to see how it currently fails.


      [bugs:#2987] gcc-torture-execute-strlen-4.c fails

      Status: pending-fixed
      Created: Sun Mar 22, 2020 11:49 AM UTC by Philipp Klaus Krause
      Last Updated: Fri Dec 29, 2023 04:16 AM UTC
      Owner: Gabriele Gorla

      Now that gcc-torture-execute-strlen-4.c compiles, the test still fails when enabled.
      We need to check if that is okay (maybe it just relies on GCC implementation-defined behaviour with SDCC making different choices) or a real SDCC bug.


      Sent from sourceforge.net because you indicated interest in https://sourceforge.net/p/sdcc/bugs/2987/

      To unsubscribe from further messages, please visit https://sourceforge.net/auth/subscriptions/

       

      Related

      Bugs: #2987
      Commit: [r14561]


Log in to post a comment.

MongoDB Logo MongoDB