Ticket #4002 (closed feature request: fixed)

Opened 3 years ago

Last modified 17 months ago

Base new top-level names on the binder the code originated from (if any)

Reported by: tibbe Owned by:
Priority: normal Milestone: 7.4.1
Component: Compiler Version: 6.12.2
Keywords: Cc:
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: None/Unknown Difficulty: Unknown
Test Case: Blocked By:
Blocking: Related Tickets:

Description

Generated top-level names currently starts with a followed by some unique string. It would be nice if it the new name was based on the binder the code originated from, making the Core easier to read.

In ticket #3693 batterseapower made such a change in an unrelated patch.  http://hackage.haskell.org/trac/ghc/attachment/ticket/3693/StackTraces.dpatch

If someone has a good suggestion for naming I could try to implement this. How about changing <a>_<unique string> to <originating binder>_<unique string>?

Change History

Changed 3 years ago by simonmar

We should try to understand the relationship between Max's change and what the tidier currently does, becuase I think there's some overlap. In the tidier we assign names to exposed identifiers by prefixing the name of the enclosing top-level exported function.

Changed 3 years ago by igloo

  • milestone set to 6.14.1

Changed 2 years ago by igloo

  • milestone changed from 7.0.1 to 7.0.2

Changed 2 years ago by igloo

  • milestone changed from 7.0.2 to 7.2.1

Changed 20 months ago by igloo

  • milestone changed from 7.2.1 to 7.4.1

Changed 17 months ago by simonmar

  • status changed from new to closed
  • difficulty set to Unknown
  • resolution set to fixed

We've been doing this for some time now.

Note: See TracTickets for help on using tickets.