[GHC] #8607: Invalid location reported for type constructors

[GHC]

#8607: Invalid location reported for type constructors
-------------------------------------+------------------------------------
        Reporter:  edsko             |            Owner:  goldfire
            Type:  bug               |           Status:  new
        Priority:  normal            |        Milestone:
       Component:  Compiler          |          Version:  7.7
      Resolution:                    |         Keywords:
Operating System:  Unknown/Multiple  |     Architecture:  Unknown/Multiple
 Type of failure:  None/Unknown      |       Difficulty:  Unknown
       Test Case:                    |       Blocked By:
        Blocking:                    |  Related Tickets:
-------------------------------------+------------------------------------

Comment (by edsko):

 Well I'm not familiar at all with this code, so I may be talking nonsense;
 I just wanted to point out that a `Located Name` is _not_ redundant: a
 `Located Name` contains two locations, and that's exactly right: one if
 the location of the identifier, and the other is the def span of whatever
 the identifier is referring to. The change in this commit changed _both_
 of those locations to be the span of the entire type declaration; it seems
 to be that it ought to be sufficient to change only the `nameSrcSpan` but
 not the location of the identifier itself.

--
Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/8607#comment:9>
GHC <http://www.haskell.org/ghc/>
The Glasgow Haskell Compiler