[GHC] #8170: Remove _scc_

#8170: Remove _scc_ ------------------------------------+------------------------------------- Reporter: monoidal | Owner: Type: bug | Status: new Priority: normal | Milestone: Component: Compiler | Version: 7.6.3 Keywords: | Operating System: Unknown/Multiple Architecture: Unknown/Multiple | Type of failure: Other Difficulty: Unknown | Test Case: Blocked By: | Blocking: Related Tickets: | ------------------------------------+------------------------------------- _scc_ is an ancient way of writing {-# SCC #-} pragmas. The old pragmas were discouraged in 2001 ([http://www.haskell.org/ghc/docs/5.00/set/release-5-00.html GHC 5.0]) and officialy deprecated in 2007 ([http://www.haskell.org/ghc/docs/6.8.3/html/users_guide/release-6-8-1.html GHC 6.8]). I downloaded the archive of the latest versions of packages in Hackage; there are over 1000 occurrences of {-# SCC #-} and 0 occurrences of _scc_. Patch attached. -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/8170 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#8170: Remove _scc_ -------------------------------------+------------------------------------ Reporter: monoidal | Owner: Type: bug | Status: patch Priority: normal | Milestone: Component: Compiler | Version: 7.6.3 Resolution: | Keywords: Operating System: Unknown/Multiple | Architecture: Unknown/Multiple Type of failure: Other | Difficulty: Unknown Test Case: | Blocked By: Blocking: | Related Tickets: -------------------------------------+------------------------------------ Changes (by monoidal): * status: new => patch -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/8170#comment:1 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#8170: Remove _scc_ -------------------------------------+------------------------------------ Reporter: monoidal | Owner: Type: bug | Status: patch Priority: normal | Milestone: Component: Compiler | Version: 7.6.3 Resolution: | Keywords: Operating System: Unknown/Multiple | Architecture: Unknown/Multiple Type of failure: Other | Difficulty: Unknown Test Case: | Blocked By: Blocking: | Related Tickets: -------------------------------------+------------------------------------ Comment (by simonpj): Looks good to me. Austin, can you do this? (Is there any documentation to change?) Simon -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/8170#comment:2 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#8170: Remove _scc_ -------------------------------------+------------------------------------ Reporter: monoidal | Owner: Type: bug | Status: patch Priority: normal | Milestone: Component: Compiler | Version: 7.6.3 Resolution: | Keywords: Operating System: Unknown/Multiple | Architecture: Unknown/Multiple Type of failure: Other | Difficulty: Unknown Test Case: | Blocked By: Blocking: | Related Tickets: -------------------------------------+------------------------------------ Comment (by thoughtpolice): Unfortunately we still use this throughout nofib looking through grep, so we'll have to fix more instances of this still. -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/8170#comment:3 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#8170: Remove _scc_ -------------------------------------+------------------------------------ Reporter: monoidal | Owner: Type: bug | Status: patch Priority: normal | Milestone: Component: Compiler | Version: 7.6.3 Resolution: | Keywords: Operating System: Unknown/Multiple | Architecture: Unknown/Multiple Type of failure: Other | Difficulty: Unknown Test Case: | Blocked By: Blocking: | Related Tickets: -------------------------------------+------------------------------------ Comment (by simonpj): It can't take long to change the nofib uses, can it? -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/8170#comment:4 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#8170: Remove _scc_ -------------------------------------+------------------------------------ Reporter: monoidal | Owner: Type: bug | Status: patch Priority: normal | Milestone: Component: Compiler | Version: 7.6.3 Resolution: | Keywords: Operating System: Unknown/Multiple | Architecture: Unknown/Multiple Type of failure: Other | Difficulty: Unknown Test Case: | Blocked By: Blocking: | Related Tickets: -------------------------------------+------------------------------------ Comment (by thoughtpolice): No, of course. I just wanted to note it here since I put it aside while merging patches last night and wanted to keep track of why. I'll step through this again sometime tonight and get it cleaned up. -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/8170#comment:5 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#8170: Remove _scc_ -------------------------------------+------------------------------------ Reporter: monoidal | Owner: Type: bug | Status: patch Priority: normal | Milestone: Component: Compiler | Version: 7.6.3 Resolution: | Keywords: Operating System: Unknown/Multiple | Architecture: Unknown/Multiple Type of failure: Other | Difficulty: Unknown Test Case: | Blocked By: Blocking: | Related Tickets: -------------------------------------+------------------------------------ Comment (by monoidal): Sorry about that; I didn't think about nofib. _scc_ occurs in 4 files; only one is not bitrotten, and it's a Happy file (no actual pragma, only its parsing). If you want to minimize work, I would not do anything with nofib. Alternative: I attached a 1MB patch file that removes the pragma in the files. It's so huge because nofib keeps a corresponding .hs file for .y file, and a single change there has many consequences. Either way, please don't lose too much time on this. -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/8170#comment:6 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#8170: Remove _scc_
-------------------------------------+------------------------------------
Reporter: monoidal | Owner:
Type: bug | Status: patch
Priority: normal | Milestone:
Component: Compiler | Version: 7.6.3
Resolution: | Keywords:
Operating System: Unknown/Multiple | Architecture: Unknown/Multiple
Type of failure: Other | Difficulty: Unknown
Test Case: | Blocked By:
Blocking: | Related Tickets:
-------------------------------------+------------------------------------
Comment (by Krzysztof Gogolewski

#8170: Remove _scc_
-------------------------------------+------------------------------------
Reporter: monoidal | Owner:
Type: bug | Status: patch
Priority: normal | Milestone:
Component: Compiler | Version: 7.6.3
Resolution: | Keywords:
Operating System: Unknown/Multiple | Architecture: Unknown/Multiple
Type of failure: Other | Difficulty: Unknown
Test Case: | Blocked By:
Blocking: | Related Tickets:
-------------------------------------+------------------------------------
Comment (by Krzysztof Gogolewski

#8170: Remove _scc_
-------------------------------------+------------------------------------
Reporter: monoidal | Owner:
Type: bug | Status: patch
Priority: normal | Milestone:
Component: Compiler | Version: 7.6.3
Resolution: | Keywords:
Operating System: Unknown/Multiple | Architecture: Unknown/Multiple
Type of failure: Other | Difficulty: Unknown
Test Case: | Blocked By:
Blocking: | Related Tickets:
-------------------------------------+------------------------------------
Comment (by Krzysztof Gogolewski

#8170: Remove _scc_ -------------------------------------+------------------------------------ Reporter: monoidal | Owner: Type: bug | Status: closed Priority: normal | Milestone: Component: Compiler | Version: 7.6.3 Resolution: fixed | Keywords: Operating System: Unknown/Multiple | Architecture: Unknown/Multiple Type of failure: Other | Difficulty: Unknown Test Case: | Blocked By: Blocking: | Related Tickets: -------------------------------------+------------------------------------ Changes (by monoidal): * status: patch => closed * resolution: => fixed Comment: I've applied the changes, except for the megapatch to one happy-generated file which seemed pointless. (Austin consented to me finishing the ticket.) -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/8170#comment:10 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#8170: Remove _scc_
-------------------------------------+------------------------------------
Reporter: monoidal | Owner:
Type: bug | Status: closed
Priority: normal | Milestone:
Component: Compiler | Version: 7.6.3
Resolution: fixed | Keywords:
Operating System: Unknown/Multiple | Architecture: Unknown/Multiple
Type of failure: Other | Difficulty: Unknown
Test Case: | Blocked By:
Blocking: | Related Tickets:
-------------------------------------+------------------------------------
Comment (by Krzysztof Gogolewski
participants (1)
-
GHC