[Git][ghc/ghc][wip/driver-fix-defer] driver: Fix -fdefer-diagnostics flag
Matthew Pickering (@mpickering)
gitlab at gitlab.haskell.org
Tue Nov 1 13:52:34 UTC 2022
Matthew Pickering pushed to branch wip/driver-fix-defer at Glasgow Haskell Compiler / GHC
Commits:
9de1dc50 by Matthew Pickering at 2022-11-01T13:52:24+00:00
driver: Fix -fdefer-diagnostics flag
The `withDeferredDiagnostics` wrapper wasn't doing anything because the
session it was modifying wasn't used in hsc_env. Therefore the fix is
simple, just push the `getSession` call into the scope of
`withDeferredDiagnostics`.
Fixes #22391
- - - - -
9 changed files:
- compiler/GHC/Driver/Make.hs
- + testsuite/tests/driver/t22391/Makefile
- + testsuite/tests/driver/t22391/all.T
- + testsuite/tests/driver/t22391/src/Lib.hs
- + testsuite/tests/driver/t22391/src/Lib/A.hs
- + testsuite/tests/driver/t22391/src/Lib/B.hs
- + testsuite/tests/driver/t22391/t22391.stderr
- + testsuite/tests/driver/t22391/t22391j.stderr
- testsuite/tests/ghci/prog018/prog018.stdout
Changes:
=====================================
compiler/GHC/Driver/Make.hs
=====================================
@@ -741,8 +741,8 @@ load' mhmi_cache how_much mHscMessage mod_graph = do
Just n -> return n
setSession $ hscUpdateHUG (unitEnv_map pruneHomeUnitEnv) hsc_env
- hsc_env <- getSession
- (upsweep_ok, hsc_env1) <- withDeferredDiagnostics $
+ (upsweep_ok, hsc_env1) <- withDeferredDiagnostics $ do
+ hsc_env <- getSession
liftIO $ upsweep n_jobs hsc_env mhmi_cache mHscMessage (toCache pruned_cache) build_plan
setSession hsc_env1
case upsweep_ok of
=====================================
testsuite/tests/driver/t22391/Makefile
=====================================
@@ -0,0 +1,19 @@
+TOP=../../..
+include $(TOP)/mk/boilerplate.mk
+include $(TOP)/mk/test.mk
+
+warnings-ghc-deferred: clean
+ "$GHC" --make -Wall -fdefer-diagnostics src/Lib.hs src/Lib/A.hs src/Lib/B.hs"
+ ghc --version
+
+warnings-ghc-regular: clean
+ bash -c "ghc --make -Wall src/Lib.hs src/Lib/A.hs src/Lib/B.hs"
+ ghc --version
+
+.PHONY: warnings-ghc
+
+clean:
+ rm -rf src/**/*.{hi,o}
+ rm -rf **/*.{hi,o}
+
+.PHONY: clean
=====================================
testsuite/tests/driver/t22391/all.T
=====================================
@@ -0,0 +1,5 @@
+test('t22391', [extra_files(['src'])],
+ multimod_compile, ['Lib', '-v1 -Wall -fhide-source-paths -isrc -fdefer-diagnostics'])
+
+test('t22391j', [extra_files(['src'])],
+ multimod_compile, ['Lib', '-v1 -Wall -fhide-source-paths -isrc -fdefer-diagnostics -j2'])
=====================================
testsuite/tests/driver/t22391/src/Lib.hs
=====================================
@@ -0,0 +1,11 @@
+module Lib
+ ( someFunc
+ ) where
+
+import Lib.A
+import Lib.B
+
+blah = 3
+
+someFunc :: IO ()
+someFunc = putStrLn "someFunc"
=====================================
testsuite/tests/driver/t22391/src/Lib/A.hs
=====================================
@@ -0,0 +1,3 @@
+module Lib.A where
+
+blast = 1
=====================================
testsuite/tests/driver/t22391/src/Lib/B.hs
=====================================
@@ -0,0 +1,3 @@
+module Lib.B where
+
+warnmeup = 4
=====================================
testsuite/tests/driver/t22391/t22391.stderr
=====================================
@@ -0,0 +1,43 @@
+[1 of 3] Compiling Lib.A
+[2 of 3] Compiling Lib.B
+[3 of 3] Compiling Lib
+
+src/Lib/A.hs:3:1: warning: [GHC-38417] [-Wmissing-signatures (in -Wall)]
+ Top-level binding with no type signature: blast :: Integer
+
+src/Lib/A.hs:3:9: warning: [GHC-18042] [-Wtype-defaults (in -Wall)]
+ • Defaulting the type variable ‘a0’ to type ‘Integer’ in the following constraint
+ Num a0 arising from the literal ‘1’
+ • In the expression: 1
+ In an equation for ‘blast’: blast = 1
+
+src/Lib/B.hs:3:1: warning: [GHC-38417] [-Wmissing-signatures (in -Wall)]
+ Top-level binding with no type signature: warnmeup :: Integer
+
+src/Lib/B.hs:3:12: warning: [GHC-18042] [-Wtype-defaults (in -Wall)]
+ • Defaulting the type variable ‘a0’ to type ‘Integer’ in the following constraint
+ Num a0 arising from the literal ‘4’
+ • In the expression: 4
+ In an equation for ‘warnmeup’: warnmeup = 4
+
+src/Lib.hs:5:1: warning: [-Wunused-imports (in -Wextra)]
+ The import of ‘Lib.A’ is redundant
+ except perhaps to import instances from ‘Lib.A’
+ To import instances alone, use: import Lib.A()
+
+src/Lib.hs:6:1: warning: [-Wunused-imports (in -Wextra)]
+ The import of ‘Lib.B’ is redundant
+ except perhaps to import instances from ‘Lib.B’
+ To import instances alone, use: import Lib.B()
+
+src/Lib.hs:8:1: warning: [GHC-38417] [-Wmissing-signatures (in -Wall)]
+ Top-level binding with no type signature: blah :: Integer
+
+src/Lib.hs:8:1: warning: [-Wunused-top-binds (in -Wextra, -Wunused-binds)]
+ Defined but not used: ‘blah’
+
+src/Lib.hs:8:8: warning: [GHC-18042] [-Wtype-defaults (in -Wall)]
+ • Defaulting the type variable ‘a0’ to type ‘Integer’ in the following constraint
+ Num a0 arising from the literal ‘3’
+ • In the expression: 3
+ In an equation for ‘blah’: blah = 3
=====================================
testsuite/tests/driver/t22391/t22391j.stderr
=====================================
@@ -0,0 +1,43 @@
+[1 of 3] Compiling Lib.A
+[2 of 3] Compiling Lib.B
+[3 of 3] Compiling Lib
+
+src/Lib/A.hs:3:1: warning: [GHC-38417] [-Wmissing-signatures (in -Wall)]
+ Top-level binding with no type signature: blast :: Integer
+
+src/Lib/A.hs:3:9: warning: [GHC-18042] [-Wtype-defaults (in -Wall)]
+ • Defaulting the type variable ‘a0’ to type ‘Integer’ in the following constraint
+ Num a0 arising from the literal ‘1’
+ • In the expression: 1
+ In an equation for ‘blast’: blast = 1
+
+src/Lib/B.hs:3:1: warning: [GHC-38417] [-Wmissing-signatures (in -Wall)]
+ Top-level binding with no type signature: warnmeup :: Integer
+
+src/Lib/B.hs:3:12: warning: [GHC-18042] [-Wtype-defaults (in -Wall)]
+ • Defaulting the type variable ‘a0’ to type ‘Integer’ in the following constraint
+ Num a0 arising from the literal ‘4’
+ • In the expression: 4
+ In an equation for ‘warnmeup’: warnmeup = 4
+
+src/Lib.hs:5:1: warning: [-Wunused-imports (in -Wextra)]
+ The import of ‘Lib.A’ is redundant
+ except perhaps to import instances from ‘Lib.A’
+ To import instances alone, use: import Lib.A()
+
+src/Lib.hs:6:1: warning: [-Wunused-imports (in -Wextra)]
+ The import of ‘Lib.B’ is redundant
+ except perhaps to import instances from ‘Lib.B’
+ To import instances alone, use: import Lib.B()
+
+src/Lib.hs:8:1: warning: [GHC-38417] [-Wmissing-signatures (in -Wall)]
+ Top-level binding with no type signature: blah :: Integer
+
+src/Lib.hs:8:1: warning: [-Wunused-top-binds (in -Wextra, -Wunused-binds)]
+ Defined but not used: ‘blah’
+
+src/Lib.hs:8:8: warning: [GHC-18042] [-Wtype-defaults (in -Wall)]
+ • Defaulting the type variable ‘a0’ to type ‘Integer’ in the following constraint
+ Num a0 arising from the literal ‘3’
+ • In the expression: 3
+ In an equation for ‘blah’: blah = 3
=====================================
testsuite/tests/ghci/prog018/prog018.stdout
=====================================
@@ -1,4 +1,6 @@
[1 of 3] Compiling A ( A.hs, interpreted )
+[2 of 3] Compiling B ( B.hs, interpreted )
+[3 of 3] Compiling C ( C.hs, interpreted )
A.hs:5:1: warning: [GHC-62161] [-Wincomplete-patterns (in -Wextra)]
Pattern match(es) are non-exhaustive
@@ -7,19 +9,14 @@ A.hs:5:1: warning: [GHC-62161] [-Wincomplete-patterns (in -Wextra)]
A.hs:8:15: warning: [-Wunused-matches (in -Wextra)]
Defined but not used: ‘x’
-[2 of 3] Compiling B ( B.hs, interpreted )
B.hs:7:1: warning: [-Wunused-imports (in -Wextra)]
The import of ‘Data.Tuple’ is redundant
except perhaps to import instances from ‘Data.Tuple’
To import instances alone, use: import Data.Tuple()
-[3 of 3] Compiling C ( C.hs, interpreted )
C.hs:6:7: error: [GHC-88464]
Variable not in scope: variableNotInScope :: ()
Failed, two modules loaded.
[3 of 3] Compiling C ( C.hs, interpreted )
-
-C.hs:6:7: error: [GHC-88464]
- Variable not in scope: variableNotInScope :: ()
Failed, two modules loaded.
View it on GitLab: https://gitlab.haskell.org/ghc/ghc/-/commit/9de1dc501777374236f9e47d6a879dfd988ab7dc
--
View it on GitLab: https://gitlab.haskell.org/ghc/ghc/-/commit/9de1dc501777374236f9e47d6a879dfd988ab7dc
You're receiving this email because of your account on gitlab.haskell.org.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.haskell.org/pipermail/ghc-commits/attachments/20221101/39d692d8/attachment-0001.html>
More information about the ghc-commits
mailing list