Opened 6 years ago

Last modified 3 months ago

#3379 new task

GHC should use the standard binary package

Reported by: igloo Owned by:
Priority: normal Milestone: 7.12.1
Component: Compiler Version: 6.10.4
Keywords: Cc: dterei
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: None/Unknown Test Case:
Blocked By: Blocking:
Related Tickets: Differential Revisions:

Description

GHC should use the standard binary package, rather than reimplementing its functionality itself. If the current binary package is slower than GHC's Binary, then we should fix that.

There's some discussion about this in #3041.

Change History (9)

comment:1 Changed 6 years ago by batterseapower

Annotations should also use the standard binary package for serialisation if it becomes a boot library.

comment:2 Changed 4 years ago by igloo

  • Milestone changed from 7.0.1 to 7.0.2

comment:3 Changed 4 years ago by igloo

  • Milestone changed from 7.0.2 to 7.2.1

comment:4 Changed 4 years ago by igloo

  • Milestone changed from 7.2.1 to 7.6.1
  • Type of failure set to None/Unknown
[16:40] < JaffaCake> Igloo: I don't know if we can use binary as it stands, we
                     have a customised Binary class that passes some state
                     around
[16:40] < copumpkin> there really needs to be a Get/Put transformer
[16:41] < JaffaCake> yep
[16:41] < JaffaCake> but also we'd have to be careful that the layers got
                     optimised away properly

comment:5 Changed 3 years ago by dterei

  • Cc dterei added

comment:6 Changed 3 years ago by igloo

  • Milestone changed from 7.6.1 to 7.6.2

comment:7 Changed 9 months ago by thoughtpolice

  • Milestone changed from 7.6.2 to 7.10.1

Moving to 7.10.1.

comment:8 Changed 6 months ago by thomie

GHC still has is its own Binary module, see compiler/utils/Binary.hs and:

$ git grep 'import Binary' compiler/

comment:9 Changed 3 months ago by thoughtpolice

  • Milestone changed from 7.10.1 to 7.12.1

Moving to 7.12.1 milestone; if you feel this is an error and should be addressed sooner, please move it back to the 7.10.1 milestone.

Note: See TracTickets for help on using tickets.