[GHC] #10511: Simplify some preprocessor expressions
#10511: Simplify some preprocessor expressions -------------------------------------+------------------------------------- Reporter: Berdes | Owner: Berdes Type: task | Status: new Priority: low | Milestone: Component: Compiler | Version: 7.10.1 Keywords: | Operating System: Unknown/Multiple Architecture: | Type of failure: None/Unknown Unknown/Multiple | Blocked By: Test Case: | Related Tickets: Blocking: | Differential Revisions: | -------------------------------------+------------------------------------- Some proprocessor expressions can be simplified. For exemple, when `_MSC_VER` is defined, `_WIN32` is also defined. So `#if defined(_MSC_VER) || defined(_WIN32)` can be simplified to `#if defined(_WIN32)`. -- Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/10511> GHC <http://www.haskell.org/ghc/> The Glasgow Haskell Compiler
#10511: Simplify some preprocessor expressions -------------------------------------+------------------------------------- Reporter: Berdes | Owner: Berdes Type: task | Status: patch Priority: low | Milestone: Component: Compiler | Version: 7.10.1 Resolution: | Keywords: Operating System: Unknown/Multiple | Architecture: Type of failure: None/Unknown | Unknown/Multiple Blocked By: | Test Case: Related Tickets: | Blocking: | Differential Revisions: D981 -------------------------------------+------------------------------------- Changes (by Berdes): * status: new => patch * differential: => D981 Old description:
Some proprocessor expressions can be simplified. For exemple, when `_MSC_VER` is defined, `_WIN32` is also defined.
So `#if defined(_MSC_VER) || defined(_WIN32)` can be simplified to `#if defined(_WIN32)`.
New description: Some preprocessor expressions can be simplified. For exemple, when `_MSC_VER` is defined, `_WIN32` is also defined. So `#if defined(_MSC_VER) || defined(_WIN32)` can be simplified to `#if defined(_WIN32)`. -- -- Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/10511#comment:1> GHC <http://www.haskell.org/ghc/> The Glasgow Haskell Compiler
#10511: Simplify some preprocessor expressions -------------------------------------+------------------------------------- Reporter: Berdes | Owner: Berdes Type: task | Status: patch Priority: low | Milestone: Component: Compiler | Version: 7.10.1 Resolution: | Keywords: Operating System: Unknown/Multiple | Architecture: Type of failure: None/Unknown | Unknown/Multiple Blocked By: | Test Case: Related Tickets: | Blocking: | Differential Revisions: D981 -------------------------------------+------------------------------------- Comment (by Thomas Miedema <thomasmiedema@…>): In [changeset:"ce53138ff0d156e9f229d0adab745d2d4cfaf582/ghc"]: {{{ #!CommitTicketReference repository="ghc" revision="ce53138ff0d156e9f229d0adab745d2d4cfaf582" Delete _MSC_VER when not necessary, fix #10511 Simplify some preprocessor expressions involving `_MSC_VER` because `_WIN32` is always defined when `_MSC_VER` is. Differential Revision: https://phabricator.haskell.org/D981 }}} -- Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/10511#comment:2> GHC <http://www.haskell.org/ghc/> The Glasgow Haskell Compiler
#10511: Simplify some preprocessor expressions -------------------------------------+------------------------------------- Reporter: Berdes | Owner: Berdes Type: task | Status: closed Priority: low | Milestone: 7.10.2 Component: Compiler | Version: 7.10.1 Resolution: fixed | Keywords: Operating System: Unknown/Multiple | Architecture: Type of failure: None/Unknown | Unknown/Multiple Blocked By: | Test Case: Related Tickets: | Blocking: | Differential Revisions: D981 -------------------------------------+------------------------------------- Changes (by thomie): * status: patch => closed * resolution: => fixed * milestone: => 7.10.2 Comment: Thanks. Feel free to cleanup other things if you see them. First discuss if it would be a very large patch, since we prefer to do those only if they have significant benefit. The problem is that they make it more difficult to use 'git blame' effectively. -- Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/10511#comment:3> GHC <http://www.haskell.org/ghc/> The Glasgow Haskell Compiler
#10511: Simplify some preprocessor expressions -------------------------------------+------------------------------------- Reporter: Berdes | Owner: Berdes Type: task | Status: closed Priority: low | Milestone: 7.10.2 Component: Compiler | Version: 7.10.1 Resolution: fixed | Keywords: Operating System: Windows | Architecture: Type of failure: None/Unknown | Unknown/Multiple Blocked By: | Test Case: Related Tickets: | Blocking: | Differential Revisions: Phab:D981 -------------------------------------+------------------------------------- Changes (by thomie): * differential: D981 => Phab:D981 * os: Unknown/Multiple => Windows -- Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/10511#comment:4> GHC <http://www.haskell.org/ghc/> The Glasgow Haskell Compiler
#10511: Simplify some preprocessor expressions -------------------------------------+------------------------------------- Reporter: Berdes | Owner: Berdes Type: task | Status: closed Priority: low | Milestone: 7.12.1 Component: Compiler | Version: 7.10.1 Resolution: fixed | Keywords: Operating System: Windows | Architecture: Type of failure: None/Unknown | Unknown/Multiple Blocked By: | Test Case: Related Tickets: | Blocking: | Differential Revisions: Phab:D981 -------------------------------------+------------------------------------- Changes (by thomie): * milestone: 7.10.2 => 7.12.1 -- Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/10511#comment:5> GHC <http://www.haskell.org/ghc/> The Glasgow Haskell Compiler
participants (1)
-
GHC