Opened 10 years ago

Closed 8 years ago

#4002 closed feature request (fixed)

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 Test Case:
Blocked By: Blocking:
Related Tickets: Differential Rev(s):
Wiki Page:

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 (6)

comment:1 Changed 10 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.

comment:2 Changed 10 years ago by igloo

Milestone: 6.14.1

comment:3 Changed 9 years ago by igloo

Milestone: 7.0.17.0.2

comment:4 Changed 9 years ago by igloo

Milestone: 7.0.27.2.1

comment:5 Changed 8 years ago by igloo

Milestone: 7.2.17.4.1

comment:6 Changed 8 years ago by simonmar

difficulty: Unknown
Resolution: fixed
Status: newclosed

We've been doing this for some time now.

Note: See TracTickets for help on using tickets.