ghc.git
3 years agoFix DeriveAnyClass (Trac #9968)
Simon Peyton Jones [Wed, 9 Dec 2015 09:07:46 +0000 (09:07 +0000)] 
Fix DeriveAnyClass (Trac #9968)

The main issue concerned things like

   data T a = MkT a deriving( C Int )

which is supposed to generate

   instance C Int (T a) where {}

But the 'Int' argument (called cls_tys in the code) wasn't
even being passed to inferConstraints and mk_data_eqn, so it
really had no chance.   DeriveAnyClass came along after this
code was written!

Anyway I did quite a bit of tidying up in inferConstraints.

Also I discovered that this case was not covered at all

   data T a b = MkT a b deriving( Bifunctor )

What constraints should we generate for the instance context?
We can deal with classes whose last arg has kind *, like Eq, Ord;
or (* -> *), like Functor, Traversable.  But we really don't have
a story for classes whose last arg has kind (* -> * -> *).

So I augmented checkSideConditions to check for that and give
a sensible error message.

ToDo: update the user manual.

3 years agoAdd missing whitespace in toArgs' error msg
Herbert Valerio Riedel [Tue, 8 Dec 2015 22:01:35 +0000 (23:01 +0100)] 
Add missing whitespace in toArgs' error msg

Differential Revision: https://phabricator.haskell.org/D1591

3 years agoRefactor GHCi Command type; allow "hidden" commands
Herbert Valerio Riedel [Tue, 8 Dec 2015 21:59:41 +0000 (22:59 +0100)] 
Refactor GHCi Command type; allow "hidden" commands

This transforms the 'Command' tuple into a record which is
easier to extend.

While at it, this refactoring turns the IDE `:complete` into a hidden
command excluded from completion.

The next obvious step is to add a summary text field for constructing
the `:help` output (as well as allowing to get `:help <CMD>` for single
commands.

This is a preparatory refactoring for D1240 / #10874

Reviewed By: thomie, bgamari

Differential Revision: https://phabricator.haskell.org/D1590

3 years agoAn assortment of typos
Gabor Greif [Tue, 8 Dec 2015 15:43:34 +0000 (16:43 +0100)] 
An assortment of typos

3 years agoComments about polymorphic recursion
Simon Peyton Jones [Tue, 8 Dec 2015 13:11:42 +0000 (13:11 +0000)] 
Comments about polymorphic recursion

See Trac #11176

3 years agoComments only
Simon Peyton Jones [Mon, 7 Dec 2015 12:48:44 +0000 (12:48 +0000)] 
Comments only

3 years agousers_guide: Show sub-sub-sections in ToC
Ben Gamari [Tue, 8 Dec 2015 14:59:43 +0000 (15:59 +0100)] 
users_guide: Show sub-sub-sections in ToC

3 years agodocs/glasgow_exts: Use warning admonition
Ben Gamari [Tue, 8 Dec 2015 12:14:14 +0000 (13:14 +0100)] 
docs/glasgow_exts: Use warning admonition

3 years agoFix typo sneaked in with fd3b845c01aa26b6e5
Herbert Valerio Riedel [Tue, 8 Dec 2015 13:54:14 +0000 (14:54 +0100)] 
Fix typo sneaked in with fd3b845c01aa26b6e5

3 years agoFix double MaybeT instance
Herbert Valerio Riedel [Tue, 8 Dec 2015 10:45:38 +0000 (11:45 +0100)] 
Fix double MaybeT instance

This is a fixup to fd3b845c01aa26b6e5cd12c00af59e5468e21b1b
which didn't take into account 09333313f32be975faf9158fcd3648489d78ad82
having pushed as well.

3 years agoRename s/7.12.1/8.0.1/ two minor occurences
Herbert Valerio Riedel [Tue, 8 Dec 2015 10:32:48 +0000 (11:32 +0100)] 
Rename s/7.12.1/8.0.1/ two minor occurences

[skip ci]

3 years agoUpdate libffi-tarballs submodule to libffi 3.1 (re #10238)
Herbert Valerio Riedel [Tue, 8 Dec 2015 09:46:04 +0000 (10:46 +0100)] 
Update libffi-tarballs submodule to libffi 3.1 (re #10238)

Reviewers: bgamari, austin

Subscribers: thomie

Differential Revision: https://phabricator.haskell.org/D1589

GHC Trac Issues: #10238

3 years agoMake HasDynFlags more transformers friendly
Herbert Valerio Riedel [Tue, 8 Dec 2015 10:11:11 +0000 (11:11 +0100)] 
Make HasDynFlags more transformers friendly

Ideally, we'd have the more general

    instance (MonadTrans t, Monad m, HasDynFlags m) => HasDynFlags (t m) where
        getDynFlags = lift getDynFlags

definition. However, that one would overlap with the `HasDynFlags (GhcT m)`
instance. Instead we define instances for a couple of common Monad
transformers explicitly in order to avoid nasty overlapping instances.

This is a preparatory refactoring for #10874

Reviewed By: austin

Differential Revision: https://phabricator.haskell.org/D1581

3 years agoAssociate ErrorCall pattern with ErrorCall type
Herbert Valerio Riedel [Tue, 8 Dec 2015 10:09:48 +0000 (11:09 +0100)] 
Associate ErrorCall pattern with ErrorCall type

This way,

    import Control.Exception (ErrorCall(ErrorCall))

or

    import Control.Exception (ErrorCall(..))

work as expected, and import the `ErrorCall` compatibility pattern as well.

When #5273 was implemented, it wasn't possible yet to associated
patterns with their types (see #10653).

Reviewed By: austin

Differential Revision: https://phabricator.haskell.org/D1588

3 years agoIntroduce HasGhciState class and refactor use-sites
Herbert Valerio Riedel [Tue, 8 Dec 2015 07:48:21 +0000 (08:48 +0100)] 
Introduce HasGhciState class and refactor use-sites

This allows to reach the GhciState without having to keep
track how many Monad transformer layers sit on top of the
GHCi monad.

While at it, this also refactors code to make more use of the
existing `modifyGHCiState` operation.

This is a preparatory refactoring for #10874

Differential Revision: https://phabricator.haskell.org/D1582

3 years agoGet rid of tcView altogether
Ömer Sinan Ağacan [Mon, 7 Dec 2015 16:41:51 +0000 (11:41 -0500)] 
Get rid of tcView altogether

This is just a trivial renaming that implements a ToDo mentioned in a comment
in Type.hs.

Adding Simon as reviewer since he added the ToDo comment.

Reviewers: simonpj, austin, goldfire, bgamari

Subscribers: thomie

Differential Revision: https://phabricator.haskell.org/D1584

3 years agoRemove redundant CPP conditionals
Herbert Valerio Riedel [Mon, 7 Dec 2015 21:50:44 +0000 (22:50 +0100)] 
Remove redundant CPP conditionals

It makes little sense to have __GLASGOW_HASKELL__ conditional
code inside GHCi's source-code, as GHCi is only ever build
by the current stage1 GHC, whose version is assumed to be the
same as the GHCi version being built.

3 years agoRe-use `transformers`'s `MaybeT` rather than our own
Herbert Valerio Riedel [Mon, 7 Dec 2015 16:32:23 +0000 (17:32 +0100)] 
Re-use `transformers`'s `MaybeT` rather than our own

The now removed `MaybeT` type was originally added back in 2008
via bc845b714132a897032502536fea8cd018ce325b

Reviewed By: bgamari

Differential Revision: https://phabricator.haskell.org/D1583

3 years agorts: One more Clang-unfriendly CPP usage
Ben Gamari [Mon, 7 Dec 2015 13:08:28 +0000 (14:08 +0100)] 
rts: One more Clang-unfriendly CPP usage

3 years agoghc-pkg: don't sort packages unnecessarily
Thomas Miedema [Mon, 7 Dec 2015 12:22:03 +0000 (13:22 +0100)] 
ghc-pkg: don't sort packages unnecessarily

The packages in the package database are already sorted alphabetically
by this point (see db_stack_sorted).

This is a better fix for #8245, commit 021b1f8.

Test Plan: look at output of './inplace/bin/ghc-pkg list
[--simple-output]'

Reviewers: austin, bgamari, psibi

Reviewed By: psibi

Differential Revision: https://phabricator.haskell.org/D1579

GHC Trac Issues: #8245

3 years agoMove checking for missing signatures to RnNames.reportUnusedNames
Eric Seidel [Mon, 7 Dec 2015 11:42:38 +0000 (12:42 +0100)] 
Move checking for missing signatures to RnNames.reportUnusedNames

Checking for missing signatures before renaming the export list is
prone to errors, so we now perform the check in `reportUnusedNames` at
which point everything has been renamed.

Test Plan: validate, new test case is T10908

Reviewers: goldfire, simonpj, austin, bgamari

Subscribers: thomie

Projects: #ghc

Differential Revision: https://phabricator.haskell.org/D1561

GHC Trac Issues: #10908

3 years agoRefactor ConDecl
Alan Zimmerman [Mon, 7 Dec 2015 11:40:38 +0000 (12:40 +0100)] 
Refactor ConDecl

The ConDecl type in HsDecls is an uneasy compromise. For the most part,
HsSyn directly reflects the syntax written by the programmer; and that
gives just the right "pegs" on which to hang Alan's API annotations. But
ConDecl doesn't properly reflect the syntax of Haskell-98 and GADT-style
data type declarations.

To be concrete, here's a draft new data type

```lang=hs
data ConDecl name
  | ConDeclGADT
      { con_names   :: [Located name]
      , con_type    :: LHsSigType name  -- The type after the ‘::’
      , con_doc     :: Maybe LHsDocString }

  | ConDeclH98
      { con_name    :: Located name

      , con_qvars     :: Maybe (LHsQTyVars name)
        -- User-written forall (if any), and its implicit
        -- kind variables
        -- Non-Nothing needs -XExistentialQuantification

      , con_cxt       :: Maybe (LHsContext name)
        -- ^ User-written context (if any)

      , con_details   :: HsConDeclDetails name
          -- ^ Arguments

      , con_doc       :: Maybe LHsDocString
          -- ^ A possible Haddock comment.
      } deriving (Typeable)
```

Note that

    For GADTs, just keep a type. That's what the user writes.
    NB:HsType can represent records on the LHS of an arrow:

      { x:Int,y:Bool} -> T

    con_qvars and con_cxt are both Maybe because they are both
    optional (the forall and the context of an existential data type

    For ConDeclGADT the type variables of the data type do not scope
    over the con_type; whereas for ConDeclH98 they do scope over con_cxt
    and con_details.

Updates haddock submodule.

Test Plan: ./validate

Reviewers: simonpj, erikd, hvr, goldfire, austin, bgamari

Subscribers: erikd, goldfire, thomie, mpickering

Differential Revision: https://phabricator.haskell.org/D1558

GHC Trac Issues: #11028

3 years agoUse TypeLits in the meta-data encoding of GHC.Generics
RyanGlScott [Mon, 7 Dec 2015 11:37:50 +0000 (12:37 +0100)] 
Use TypeLits in the meta-data encoding of GHC.Generics

Test Plan: Validate.

Reviewers: simonpj, goldfire, hvr, dreixel, kosmikus, austin, bgamari

Reviewed By: kosmikus, austin, bgamari

Subscribers: RyanGlScott, Fuuzetsu, bgamari, thomie, carter, dreixel

Differential Revision: https://phabricator.haskell.org/D493

GHC Trac Issues: #9766

3 years agorts: Remove space before argument list in ASSERTs
Ben Gamari [Mon, 7 Dec 2015 10:33:06 +0000 (11:33 +0100)] 
rts: Remove space before argument list in ASSERTs

Test Plan: Validate

Reviewers: austin, erikd

Subscribers: thomie

Differential Revision: https://phabricator.haskell.org/D1569

3 years agoAdd isImport, isDecl, and isStmt functions to GHC API
Roman Shatsov [Mon, 7 Dec 2015 10:24:36 +0000 (11:24 +0100)] 
Add isImport, isDecl, and isStmt functions to GHC API

Reviewers: austin, thomie, bgamari

Reviewed By: thomie, bgamari

Subscribers: mpickering, thomie

Differential Revision: https://phabricator.haskell.org/D1518

GHC Trac Issues: #9015

3 years agoMinor stylistic fixes in glasgow_exts.rst
Ben Gamari [Mon, 7 Dec 2015 10:23:50 +0000 (11:23 +0100)] 
Minor stylistic fixes in glasgow_exts.rst

3 years agoRe-export data family when exporting a data instance without an export list
David Kraeutmann [Mon, 7 Dec 2015 10:19:28 +0000 (11:19 +0100)] 
Re-export data family when exporting a data instance without an export list

Whenever a data instance is exported, the corresponding data family
is exported, too. This allows one to write

```
     -- Foo.hs
     module Foo where

     data family T a

     -- Bar.hs
     module Bar where

     import Foo

     data instance T Int = MkT

     -- Baz.hs
     module Baz where

     import Bar (T(MkT))
```

In previous versions of GHC, this required a workaround
explicit export list in `Bar`.

Reviewers: bgamari, goldfire, austin

Reviewed By: bgamari, goldfire

Subscribers: goldfire, thomie

Differential Revision: https://phabricator.haskell.org/D1573

GHC Trac Issues: #11164

3 years agoghc-pkg: Restore old behavior in colored version; fixes 6119
Sibi Prabakaran [Mon, 7 Dec 2015 10:16:15 +0000 (11:16 +0100)] 
ghc-pkg: Restore old behavior in colored version; fixes 6119

The behavior is changed to this:
```
ghc-pkg list blahblah
/home/sibi/ghc/inplace/lib/package.conf.d
    (no packages)
```
instead of:
```
ghc-pkg list blahblah
/home/sibi/ghc/inplace/lib/package.conf.d
```

Reviewers: austin, thomie, bgamari

Reviewed By: thomie, bgamari

Subscribers: thomie

Differential Revision: https://phabricator.haskell.org/D1575

GHC Trac Issues: #6119

3 years agoIgnore generated linter.log
RyanGlScott [Mon, 7 Dec 2015 10:14:41 +0000 (11:14 +0100)] 
Ignore generated linter.log

Reviewers: austin, hvr, bgamari

Reviewed By: bgamari

Subscribers: thomie

Differential Revision: https://phabricator.haskell.org/D1577

3 years agoDocumentation: escape characters in template-haskell Haddocks
RyanGlScott [Mon, 7 Dec 2015 10:06:23 +0000 (11:06 +0100)] 
Documentation: escape characters in template-haskell Haddocks

Reviewers: hvr, austin, goldfire, bgamari

Reviewed By: goldfire, bgamari

Subscribers: thomie

Differential Revision: https://phabricator.haskell.org/D1578

3 years agolinters/check-cpp: Don't produce debug log
Ben Gamari [Mon, 7 Dec 2015 09:53:44 +0000 (10:53 +0100)] 
linters/check-cpp: Don't produce debug log

3 years agoUpdate hoopl submodule
Jan Stolarek [Mon, 7 Dec 2015 07:08:30 +0000 (08:08 +0100)] 
Update hoopl submodule

Hoopl changes required to implement #10982

3 years agoTweak use of AC_USE_SYSTEM_EXTENSIONS
Herbert Valerio Riedel [Sun, 6 Dec 2015 12:34:34 +0000 (13:34 +0100)] 
Tweak use of AC_USE_SYSTEM_EXTENSIONS

This makes sure that `AC_USE_SYSTEM_EXTENSIONS` (which
implies `AC_PROG_CC`) is called after the
`AC_ARG_WITH([cc],,)` invocation, so that the proper
CC setting is in scope. Otherwise this can break cross-compilation.

This also needs to pull in a submodule update for `unix`

This is a follow-up commit to
7af29da05d2e5a5e311a5f73f20d0f232035973b
which hopefully fixes #11168

3 years agoImplement new `-fwarn-noncanonical-monoid-instances`
Herbert Valerio Riedel [Sun, 6 Dec 2015 15:08:21 +0000 (16:08 +0100)] 
Implement new `-fwarn-noncanonical-monoid-instances`

This is similiar to the `-fwarn-noncanonical-monad-instances` warning
implemented via #11128, but applies to `Semigroup`/`Monoid` instead
and the `(<>)`/`mappend` methods (of which `mappend` is planned to move
out of `Monoid` at some point in the future being redundant and thus
error-prone).

This warning is contained in `-Wcompat` but not in `-Wall`.

This addresses #11150

Reviewed By: quchen

Differential Revision: https://phabricator.haskell.org/D1553

3 years agoMake ghc.mk compatible with pedantic /bin/sh impls
Herbert Valerio Riedel [Sun, 6 Dec 2015 10:52:35 +0000 (11:52 +0100)] 
Make ghc.mk compatible with pedantic /bin/sh impls

This fixes `for`-loops introduced via
64761ce9a899954a12d8e3ae8b400c5ad9648137
which can result in `for i in ;` which some
`/bin/sh` implementations don't like.

3 years agoAllow to compile OSMem.c when MEM_NORESERVE is not available
Herbert Valerio Riedel [Sat, 5 Dec 2015 18:15:19 +0000 (19:15 +0100)] 
Allow to compile OSMem.c when MEM_NORESERVE is not available

On some OSes such as AIX `MEM_NORESERVE` is not available.

Since this feature is only needed when the new two-step allocator (see #9706)
is enabled we can simply turn this into a runtime error to avoid a larger
refactoring of this already quite platform-sensitive code.

Reviewed By: bgamari, ezyang

Differential Revision: https://phabricator.haskell.org/D1568

3 years agoghc.mk: fix docs re-rebuilding
Sergei Trofimovich [Sat, 5 Dec 2015 10:05:54 +0000 (10:05 +0000)] 
ghc.mk: fix docs re-rebuilding

The problem:
    $ make
    <watch sphinx rebuilds docs>
    # no changes in sources
    $ make
    <watch sphinx rebuilds docs again>

The problem was due to wrong assumption about what
files exactly are generated by mkUserGuidePart.

Build system expected the following files to be created:
    docs/man/all-flags.gen.rst
    flags-recompilating-checking.gen.rst
but mkUserGuidePart generated:
    docs/users_guide/all-flags.gen.rst
    flags-recompilation-checking.gen.rst

Signed-off-by: Sergei Trofimovich <siarheit@google.com>
3 years agoghc.mk: don't run mkUserGuidePart more than once
Sergei Trofimovich [Sat, 5 Dec 2015 00:03:05 +0000 (00:03 +0000)] 
ghc.mk: don't run mkUserGuidePart more than once

When building 'html' and 'man' manuals
build system reports mkUserGuide is ran
more than once (up to 3 times in parallel).

See Note [Blessed make target file] for
more details.

Signed-off-by: Sergei Trofimovich <siarheit@google.com>
3 years agoghc.mk: cleanup: use tab consistently
Sergei Trofimovich [Fri, 4 Dec 2015 22:28:35 +0000 (22:28 +0000)] 
ghc.mk: cleanup: use tab consistently

Signed-off-by: Sergei Trofimovich <siarheit@google.com>
3 years agoEnable non-canonical Monad instance warnings for stage1/2
Herbert Valerio Riedel [Sat, 5 Dec 2015 08:59:01 +0000 (09:59 +0100)] 
Enable non-canonical Monad instance warnings for stage1/2

This makes use of the new facility introduced via
14d0f7f1221db758cd06a69f53803d9d0150164a which allows
to have certain flags passed only to the non-bootstrapping
GHC. This is needed because sometimes we can't assume the
existence of a certain flag in the bootstrapping compiler
which was only added recently to GHC HEAD.

This also updates the haddock submodule to fix a few
remaining noncanonical instance definitions.

Differential Revision: https://phabricator.haskell.org/D1571

3 years agoTemporarily disable external-json linters
Herbert Valerio Riedel [Sat, 5 Dec 2015 08:55:50 +0000 (09:55 +0100)] 
Temporarily disable external-json linters

This temporarily disables the external json linters
introduced via
020375d1e723339a95b86d0d3b8a8214b1cc144a and
901cab10609dc9795e57163834512373530fc4a5.

The new linters break the differential validation in Harbormaster
because the submodules are not cloned early enough (i.e. before `arc` is
called by the script).

3 years agoUse git.h.o copy of arcanist-external-json-linter
Herbert Valerio Riedel [Sat, 5 Dec 2015 08:15:57 +0000 (09:15 +0100)] 
Use git.h.o copy of arcanist-external-json-linter

It shouldn't have been possible to reference an external
Git submodule not hosted on git.haskell.org as we can't otherwise
ensure gitlink integrity. But it turns out the
validation hook scripts in place didn't reject
020375d1e723339a95b86d0d3b8a8214b1cc144a, so here we are...

This commit changes ghc.git to use our own fork/copy of
https://github.com/bgamari/arcanist-external-json-linter
hosted on git.haskell.org

3 years agoFix broken linters when using python3
David Kraeutmann [Sat, 5 Dec 2015 06:13:08 +0000 (00:13 -0600)] 
Fix broken linters when using python3

Reviewed By: austin

Differential Revision: https://phabricator.haskell.org/D1574

3 years agopmcheck: Comments about undecidability of literal equality
George Karachalias [Sat, 5 Dec 2015 00:52:58 +0000 (01:52 +0100)] 
pmcheck: Comments about undecidability of literal equality

3 years agopmcheck: Comments about term equality representation
George Karachalias [Sat, 5 Dec 2015 00:13:33 +0000 (01:13 +0100)] 
pmcheck: Comments about term equality representation

3 years agolibdw: enable support only on i386 and amd64
Sergei Trofimovich [Fri, 4 Dec 2015 22:07:13 +0000 (22:07 +0000)] 
libdw: enable support only on i386 and amd64

Summary:
Currently libdw requires per-arch implementation of
set_initial_registers() function.

Otherwise build fails with linkage error
(seen on sparc):

  rts/dist/build/libHSrts_thr-ghc7.11.20151129.so:
      undefined reference to `set_initial_registers'
  collect2: error: ld returned 1 exit status
  `gcc' failed in phase `Linker'. (Exit code: 1)
  make[1]: *** [ghc/stage2/build/tmp/ghc-stage2] Error 1

Converted link-time error to compile-time error,
added arch whitelist.

Signed-off-by: Sergei Trofimovich <siarheit@google.com>
Reviewers: austin, thomie, bgamari

Reviewed By: bgamari

Differential Revision: https://phabricator.haskell.org/D1567

3 years agoStgCmmMonad: Implement Outputable instance for Sequel for debugging
Ömer Sinan Ağacan [Fri, 4 Dec 2015 20:36:47 +0000 (15:36 -0500)] 
StgCmmMonad: Implement Outputable instance for Sequel for debugging

Reviewers: austin, bgamari

Reviewed By: bgamari

Subscribers: thomie

Differential Revision: https://phabricator.haskell.org/D1563

3 years agolint: Add linter to catch uses of ASSERT macro that Clang dislikes
Ben Gamari [Fri, 4 Dec 2015 12:25:26 +0000 (13:25 +0100)] 
lint: Add linter to catch uses of ASSERT macro that Clang dislikes

In particular Clang rejects uses of CPP macros where the argument list
is separated by a space from the macro name. Warn when we see ASSERT
used in this way.

3 years agoAdd linter to check for binaries accidentally added to repository
Ben Gamari [Fri, 4 Dec 2015 12:14:27 +0000 (13:14 +0100)] 
Add linter to check for binaries accidentally added to repository

This should catch mistakes like
a703fbce20969e6f02e74fee76c0a9e22b513426.

Adds an arcanist-external-json-linter submodule, which should eventually
be mirrored on haskell.org resources.

Test Plan: Validate

Reviewers: thomie, hvr, austin

Reviewed By: austin

Differential Revision: https://phabricator.haskell.org/D1515

3 years agoUpdate peak_megabytes_allocated for T9675
Ben Gamari [Fri, 4 Dec 2015 14:55:58 +0000 (09:55 -0500)] 
Update peak_megabytes_allocated for T9675

Interestingly enough this decreased with the new pattern checker. I'm
not entirely sure why at the moment as the test is merely a large
record with a bunch of selectors. I wouldn't have thought this would tax
the pattern checker particularly much but oh well.

3 years agotestsuite: Rename pmcheck/T7669 to pmcheck/T7669a
Ben Gamari [Fri, 4 Dec 2015 14:31:53 +0000 (09:31 -0500)] 
testsuite: Rename pmcheck/T7669 to pmcheck/T7669a

This was a duplicate.

3 years agoT5642: Fix skip usage
Ben Gamari [Fri, 4 Dec 2015 14:29:59 +0000 (09:29 -0500)] 
T5642: Fix skip usage

3 years agoComments only
Simon Peyton Jones [Fri, 4 Dec 2015 12:24:49 +0000 (12:24 +0000)] 
Comments only

3 years agoTidy user type errors in checkValidType
Simon Peyton Jones [Fri, 4 Dec 2015 12:23:33 +0000 (12:23 +0000)] 
Tidy user type errors in checkValidType

Trac #11144 showed that we need to tidy the type in the error message
generated in TcValidity.checkUserTypeError.

This is still unsatisfactory.  checkValidType was originally supposed
to be called only on types gotten directly from user-written HsTypes.
So its error messages do no tidying.  But TcBinds calls it checkValidType
on an /inferred/ type, which may need tidying.

Still this at least fixes the bad error message in CustomTypeErrors02,
which was the original ticket.

Some other small refactorings:

* Remove unused Kind result of getUserTypeErrorMsg
* Rename isUserErrorTy  -->   userTypeError_maybe

3 years agoMinor refactoring of user type errors
Simon Peyton Jones [Fri, 4 Dec 2015 12:11:43 +0000 (12:11 +0000)] 
Minor refactoring of user type errors

* Remove unused Kind result of getUserTypeErrorMsg
* Rename isUserErrorTy  -->   userTypeError_maybe

3 years agoFix egregious error in eta-reduction of data families
Simon Peyton Jones [Fri, 4 Dec 2015 12:08:37 +0000 (12:08 +0000)] 
Fix egregious error in eta-reduction of data families

This terrible and long-standing bug was shown up by Trac #11148.
We are trying to eta-reduce a data family instance, so that we
can then derive Functor or Generic.  But we were assuming, for
absolutely not reason whatsoever, that the type variables were
lined up in a convenient order.  The fact that it ever worked
was a fluke.

This patch fixes it properly.  Main change is in eta_reduce
in TcInstDcls.tcDataFamInstDecl

3 years agoMake -dppr-debug show contents of (TypeError ...)
Simon Peyton Jones [Fri, 4 Dec 2015 11:59:08 +0000 (11:59 +0000)] 
Make -dppr-debug show contents of (TypeError ...)

Just for debugging

3 years agoWibbles only
Simon Peyton Jones [Fri, 4 Dec 2015 11:58:05 +0000 (11:58 +0000)] 
Wibbles only

3 years agoAdd derived constraints for wildcard signatures
Simon Peyton Jones [Thu, 3 Dec 2015 13:03:07 +0000 (13:03 +0000)] 
Add derived constraints for wildcard signatures

This fixes Trac #11016

See Note [Add deriveds for signature contexts] in TcSimplify]

3 years agoCase-of-empty-alts is trivial (Trac #11155)
Simon Peyton Jones [Thu, 3 Dec 2015 12:57:54 +0000 (12:57 +0000)] 
Case-of-empty-alts is trivial (Trac #11155)

As you'll see from Trac #11155, the code generator was confused
by a binding let x = y in ....   Why did that happen? Because of
a (case y of {}) expression on the RHS.

The right thing is just to expand what a "trivial" expression is.

See Note [Empty case is trivial] in CoreUtils.

3 years agoMake callToPats deterministic in SpecConstr
Bartosz Nitka [Wed, 2 Dec 2015 14:20:55 +0000 (06:20 -0800)] 
Make callToPats deterministic in SpecConstr

This fixes a non-determinism bug where where depending on the
order of uniques allocated, the specialized workers would have different
order of arguments.

Compare:

```
  $s$wgo_s1CN :: Int# -> Int -> Int#
  [LclId, Arity=2, Str=DmdType <L,U><L,U>]
  $s$wgo_s1CN =
    \ (sc_s1CI :: Int#) (sc_s1CJ :: Int) ->
      case tagToEnum# @ Bool (<=# sc_s1CI 0#) of _ [Occ=Dead] {
        False ->
          $wgo_s1BU (Just @ Int (I# (-# sc_s1CI 1#))) (Just @ Int sc_s1CJ);
        True -> 0#
      }
```

vs

```
  $s$wgo_s18mTj :: Int -> Int# -> Int#
  [LclId, Arity=2, Str=DmdType <L,U><L,U>]
  $s$wgo_s18mTj =
    \ (sc_s18mTn :: Int) (sc_s18mTo :: Int#) ->
      case tagToEnum# @ Bool (<=# sc_s18mTo 0#) of _ [Occ=Dead] {
        False ->
          $wgo_s18mUc
            (Just @ Int (I# (-# sc_s18mTo 1#))) (Just @ Int sc_s18mTn);
        True -> 0#
      }
```

Test Plan:
I've added a new testcase
./validate

Reviewers: simonmar, simonpj, austin, goldfire, bgamari

Reviewed By: bgamari

Subscribers: thomie

Differential Revision: https://phabricator.haskell.org/D1508

GHC Trac Issues: #4012

3 years agoT5642: Skip it entirely
Ben Gamari [Fri, 4 Dec 2015 13:30:09 +0000 (14:30 +0100)] 
T5642: Skip it entirely

It uses so much memory that it would be unsafe to even allow it to run
as it may jeopardize the stability of the build-bots.

3 years agoT5642 is broken
Ben Gamari [Fri, 4 Dec 2015 12:56:33 +0000 (07:56 -0500)] 
T5642 is broken

This appears to be due to the new exhaustiveness checker. See #11163.

3 years agoBump allocations for T783
Ben Gamari [Fri, 4 Dec 2015 10:30:39 +0000 (05:30 -0500)] 
Bump allocations for T783

The new pattern match checker increased allocations by over 100%.
Tracking in #11162.

3 years agoCheck: More Clang/CPP wibbles
Ben Gamari [Fri, 4 Dec 2015 12:07:16 +0000 (13:07 +0100)] 
Check: More Clang/CPP wibbles

3 years agoUse builtin ISO 8859-1 decoder in mkTextEncoding
Herbert Valerio Riedel [Fri, 4 Dec 2015 11:15:23 +0000 (12:15 +0100)] 
Use builtin ISO 8859-1 decoder in mkTextEncoding

We already do this for UTF8/16/32, so it seems obvious do the same
for the closely related popular ISO 8859-1 encoding, and avoid iconv
issues on some platforms (such as AIX which which bundles a broken
`libiconv` by default)

This fixes #11096

3 years agoPmExpr: Fix CPP unacceptable too clang's CPP
Ben Gamari [Fri, 4 Dec 2015 10:05:06 +0000 (05:05 -0500)] 
PmExpr: Fix CPP unacceptable too clang's CPP

3 years agoOn AIX we need -D_BSD defined in <Stg.h>
Herbert Valerio Riedel [Fri, 4 Dec 2015 09:40:42 +0000 (10:40 +0100)] 
On AIX we need -D_BSD defined in <Stg.h>

As otherwise <math.h> includes <stdlib.h> which breaks compilation
of .hc files

3 years agoRTS: Rename InCall.stat struct field to .rstat
Herbert Valerio Riedel [Fri, 4 Dec 2015 09:10:55 +0000 (10:10 +0100)] 
RTS: Rename InCall.stat struct field to .rstat

On AIX, C system headers can redirect the token `stat` via

    #define stat stat64

to provide large-file support. Simply avoiding the use of `stat` as an
identifier eschews macro-replacement.

Differential Revision: https://phabricator.haskell.org/D1566

3 years agoUse Autoconf's AC_USE_SYSTEM_EXTENSIONS
Herbert Valerio Riedel [Fri, 4 Dec 2015 08:15:29 +0000 (09:15 +0100)] 
Use Autoconf's AC_USE_SYSTEM_EXTENSIONS

This takes care of setting feature test macros (i.e. let Autoconf decide when
those can be set safely) to allow subsequent Autoconf tests to better detect
available OS features.

This also includes a submodule update of unix which enables the use of
`AC_USE_SYSTEM_EXTENSIONS` in there as well.

Specifically, this takes care of setting `_GNU_SOURCE` (which allows to remove
two occurences where it's set manually) and `_ALL_SOURCE` (which fixes issues
on AIX).

See also

  https://www.gnu.org/software/autoconf/manual/autoconf-2.69/html_node/Posix-Variants.html

for details.

At some point we may want to reconsider the purpose of "rts/PosixSource.h" and
rely more on Autoconf instead.

3 years agoRemove unused import in deSugar/TmOracle.hs
George Karachalias [Fri, 4 Dec 2015 07:28:00 +0000 (08:28 +0100)] 
Remove unused import in deSugar/TmOracle.hs

3 years agoImprove performance for PM check on literals (Fixes #11160 and #11161)
George Karachalias [Fri, 4 Dec 2015 06:11:55 +0000 (07:11 +0100)] 
Improve performance for PM check on literals (Fixes #11160 and #11161)

Two changes:

1. Instead of generating constraints of the form (x ~ e) (as we do in
the paper), generate constraints of the form (e ~ e). The term oracle
(`tmOracle` in deSugar/TmOracle.hs) is not really efficient and in the
presence of many (x ~ e) constraints behaves quadratically. For
literals, constraints of the form (False ~ (x ~ lit)) are pretty common,
so if we start with { y ~ False, y ~ (x ~ lit) } we end up givng to the
solver (a) twice as many constraints as we need and (b) half of them
trigger the solver's weakness. This fixes #11160.

2. Treat two overloaded literals that look different as different. This
is not entirely correct but it is what both the previous and the current
check did. I had the ambitious plan to do the *right thing* (equality
between overloaded literals is undecidable in the general case) and just
use this assumption when issuing the warnings. It seems to generate much
more constraints than I expected (breaks #11161) so I just do it
immediately now instead of generating everything and filtering
afterwards.

Even if it is not (strictly speaking) correct, we have the following:
  * Gives the "expected" warnings (the ones Ocaml or the previous
    algorithm would give) and,
  * Most importantly, it is safe. Unless a catch-all clause exists, a
    match against literals is always non-exhaustive. So, effectively
    this affects only what is shown to the user (and, evidently,
    performance!).

3 years agoBump hoopl submodule
Ben Gamari [Thu, 3 Dec 2015 20:20:18 +0000 (15:20 -0500)] 
Bump hoopl submodule

To fix redundant patterns.

3 years agoUpdate test output
Ben Gamari [Thu, 3 Dec 2015 20:20:01 +0000 (15:20 -0500)] 
Update test output

3 years agoKill redundant patterns
Ben Gamari [Thu, 3 Dec 2015 18:43:39 +0000 (13:43 -0500)] 
Kill redundant patterns

George's new exhaustiveness checker now realizes these are impossible.
Yay!

3 years agoFix haddock syntax
Ben Gamari [Thu, 3 Dec 2015 18:43:16 +0000 (13:43 -0500)] 
Fix haddock syntax

Sadly we can't annotate the elements of a tuple

3 years agoRevert "Create empty dump files when there was nothing to dump"
Ben Gamari [Thu, 3 Dec 2015 13:59:18 +0000 (14:59 +0100)] 
Revert "Create empty dump files when there was nothing to dump"

This reverts commit 8cba907ad404ba4005558b5a8966390159938172 which
broke `-ddump-to-file`.

3 years agoextending_ghc.rst: fix broken link (Trac #10950)
Sergei Trofimovich [Thu, 3 Dec 2015 21:07:19 +0000 (21:07 +0000)] 
extending_ghc.rst: fix broken link (Trac #10950)

The error exibits as build failures
of two types:

  1. extending_ghc.rst:: ERROR:
      Anonymous hyperlink mismatch:
        1 references but 0 targets.
      See "backrefs" attribute for IDs.

  2. reading sources... [ 33%] glasgow_exts
       Exception occurred:
          pickle.dump(doctree, f, pickle.HIGHEST_PROTOCOL)
       RecursionError: maximum recursion depth exceeded
       while pickling an object

Broken link created circular reference and failed to
serialize the result.

Fixed the problem by pointing to relevant section.

Signed-off-by: Sergei Trofimovich <siarheit@google.com>
3 years agotestsuite: haddock.compiler: Bump expected allocations
Ben Gamari [Thu, 3 Dec 2015 13:44:24 +0000 (14:44 +0100)] 
testsuite: haddock.compiler: Bump expected allocations

3 years agoMajor Overhaul of Pattern Match Checking (Fixes #595)
George Karachalias [Thu, 3 Dec 2015 11:57:19 +0000 (12:57 +0100)] 
Major Overhaul of Pattern Match Checking (Fixes #595)

This patch adresses several problems concerned with exhaustiveness and
redundancy checking of pattern matching. The list of improvements includes:

* Making the check type-aware (handles GADTs, Type Families, DataKinds, etc.).
  This fixes #4139, #3927, #8970 and other related tickets.

* Making the check laziness-aware. Cases that are overlapped but affect
  evaluation are issued now with "Patterns have inaccessible right hand side".
  Additionally, "Patterns are overlapped" is now replaced by "Patterns are
  redundant".

* Improved messages for literals. This addresses tickets #5724, #2204, etc.

* Improved reasoning concerning cases where simple and overloaded
  patterns are matched (See #322).

* Substantially improved reasoning for pattern guards. Addresses #3078.

* OverloadedLists extension does not break exhaustiveness checking anymore
  (addresses #9951). Note that in general this cannot be handled but if we know
  that an argument has type '[a]', we treat it as a list since, the instance of
  'IsList' gives the identity for both 'fromList' and 'toList'. If the type is
  not clear or is not the list type, then the check cannot do much still. I am
  a bit concerned about OverlappingInstances though, since one may override the
  '[a]' instance with e.g. an '[Int]' instance that is not the identity.

* Improved reasoning for nested pattern matching (partial solution). Now we
  propagate type and (some) term constraints deeper when checking, so we can
  detect more inconsistencies. For example, this is needed for #4139.

I am still not satisfied with several things but I would like to address at
least the following before the next release:
    Term constraints are too many and not printed for non-exhaustive matches
(with the exception of literals). This sometimes results in two identical (in
appearance) uncovered warnings. Unless we actually show their difference, I
would like to have a single warning.

3 years agousers_guide/glasgow_exts.rst: fix link markup
Sergei Trofimovich [Thu, 3 Dec 2015 09:31:44 +0000 (09:31 +0000)] 
users_guide/glasgow_exts.rst: fix link markup

sphinx-1.3.1 found errors as:
  users_guide/glasgow_exts.rst:1799:
    WARNING: malformed hyperlink target.
  users_guide/glasgow_exts.rst:10638:
    WARNING: Inline interpreted text or phrase
    reference start-string without end-string.

Signed-off-by: Sergei Trofimovich <siarheit@google.com>
3 years agoUpdate bytestring submodule
Herbert Valerio Riedel [Wed, 2 Dec 2015 14:20:10 +0000 (15:20 +0100)] 
Update bytestring submodule

Differential Revision: https://phabricator.haskell.org/D1549

3 years agoErrUtils: Spruce up Haddocks
Ben Gamari [Wed, 2 Dec 2015 19:59:25 +0000 (20:59 +0100)] 
ErrUtils: Spruce up Haddocks

This is a pretty commonly needed module; Haddocks are worth the effort.

3 years agoCreate empty dump files when there was nothing to dump
Vladimir Trubilov [Wed, 2 Dec 2015 19:47:23 +0000 (20:47 +0100)] 
Create empty dump files when there was nothing to dump

This patch creates empty dump file when GHC was run with
`-ddump-rule-firings` (or `-ddump-rule-rewrites`) and `-ddump-to-file`
specified, and there were no rules applied. If dump already exists it
will be overwritten by empty one.

Test Plan: ./validate

Reviewers: austin, thomie, bgamari

Reviewed By: thomie, bgamari

Subscribers: thomie

Projects: #ghc

Differential Revision: https://phabricator.haskell.org/D1514

GHC Trac Issues: #10320

3 years agoAvoid panic due to partial ieName
Eric Seidel [Wed, 2 Dec 2015 20:37:21 +0000 (14:37 -0600)] 
Avoid panic due to partial ieName

HsImpExp.ieName is partial and fails when given e.g. `module X`

solution: use ieNames instead which returns a list of names instead of a single name.

Reviewed By: bgamari, austin

Differential Revision: https://phabricator.haskell.org/D1551

GHC Trac Issues: #11077

3 years agoRemove *.xml from gitignore
David Luposchainsky [Wed, 2 Dec 2015 20:37:07 +0000 (14:37 -0600)] 
Remove *.xml from gitignore

With the move to RST-based documentation, there is no need to ignore XML files
in the source tree anymore.

Reviewed By: bgamari

Differential Revision: https://phabricator.haskell.org/D1554

3 years agoMove Stg-specific code from DynFlags to SimplStg
Ömer Sinan Ağacan [Wed, 2 Dec 2015 20:36:56 +0000 (14:36 -0600)] 
Move Stg-specific code from DynFlags to SimplStg

Reviewed By: bgamari, austin

Differential Revision: https://phabricator.haskell.org/D1552

3 years agoStgSyn: Remove unused SRT constructor
Ömer Sinan Ağacan [Wed, 2 Dec 2015 20:35:27 +0000 (14:35 -0600)] 
StgSyn: Remove unused SRT constructor

Reviewed By: bgamari, austin

Differential Revision: https://phabricator.haskell.org/D1560

3 years agoRevert "ghci: Add support for prompt functions"
Ben Gamari [Wed, 2 Dec 2015 13:59:39 +0000 (14:59 +0100)] 
Revert "ghci: Add support for prompt functions"

This reverts commit 72e362076e7ce823678797a162d0645e088cd594 which was
accidentally merged.

3 years agoComments (TcSMonad)
Simon Peyton Jones [Wed, 2 Dec 2015 14:41:12 +0000 (14:41 +0000)] 
Comments (TcSMonad)

3 years agoComments only (isIrrefutablePat)
Simon Peyton Jones [Wed, 2 Dec 2015 14:38:42 +0000 (14:38 +0000)] 
Comments only (isIrrefutablePat)

3 years agoImplement more deterministic operations and document them
Bartosz Nitka [Wed, 2 Dec 2015 13:30:22 +0000 (05:30 -0800)] 
Implement more deterministic operations and document them

I will need them for the future determinism fixes.

Test Plan: ./validate

Reviewers: simonpj, goldfire, bgamari, austin, hvr, simonmar

Reviewed By: simonpj, simonmar

Subscribers: osa1, thomie

Differential Revision: https://phabricator.haskell.org/D1537

GHC Trac Issues: #4012

3 years agoMake the order of fixities in the iface file deterministic
Bartosz Nitka [Wed, 2 Dec 2015 11:28:13 +0000 (03:28 -0800)] 
Make the order of fixities in the iface file deterministic

This normalizes the order of written fixities by sorting by
`OccName` making it independent of `Unique` order.

Test Plan: I've added a new testcase

Reviewers: austin, bgamari, simonmar

Reviewed By: simonmar

Subscribers: thomie

Differential Revision: https://phabricator.haskell.org/D1557

GHC Trac Issues: #4012

3 years agoRefactor treatment of wildcards
Simon Peyton Jones [Tue, 1 Dec 2015 16:38:23 +0000 (17:38 +0100)] 
Refactor treatment of wildcards

This patch began as a modest refactoring of HsType and friends, to
clarify and tidy up exactly where quantification takes place in types.
Although initially driven by making the implementation of wildcards more
tidy (and fixing a number of bugs), I gradually got drawn into a pretty
big process, which I've been doing on and off for quite a long time.

There is one compiler performance regression as a result of all
this, in perf/compiler/T3064.  I still need to look into that.

* The principal driving change is described in Note [HsType binders]
  in HsType.  Well worth reading!

* Those data type changes drive almost everything else.  In particular
  we now statically know where

       (a) implicit quantification only (LHsSigType),
           e.g. in instance declaratios and SPECIALISE signatures

       (b) implicit quantification and wildcards (LHsSigWcType)
           can appear, e.g. in function type signatures

* As part of this change, HsForAllTy is (a) simplified (no wildcards)
  and (b) split into HsForAllTy and HsQualTy.  The two contructors
  appear when and only when the correponding user-level construct
  appears.  Again see Note [HsType binders].

  HsExplicitFlag disappears altogether.

* Other simplifications

     - ExprWithTySig no longer needs an ExprWithTySigOut variant

     - TypeSig no longer needs a PostRn name [name] field
       for wildcards

     - PatSynSig records a LHsSigType rather than the decomposed
       pieces

     - The mysterious 'GenericSig' is now 'ClassOpSig'

* Renamed LHsTyVarBndrs to LHsQTyVars

* There are some uninteresting knock-on changes in Haddock,
  because of the HsSyn changes

I also did a bunch of loosely-related changes:

* We already had type synonyms CoercionN/CoercionR for nominal and
  representational coercions.  I've added similar treatment for

      TcCoercionN/TcCoercionR

      mkWpCastN/mkWpCastN

  All just type synonyms but jolly useful.

* I record-ised ForeignImport and ForeignExport

* I improved the (poor) fix to Trac #10896, by making
  TcTyClsDecls.checkValidTyCl recover from errors, but adding a
  harmless, abstract TyCon to the envt if so.

* I did some significant refactoring in RnEnv.lookupSubBndrOcc,
  for reasons that I have (embarrassingly) now totally forgotten.
  It had to do with something to do with import and export

Updates haddock submodule.

3 years agoMake the determinism tests more robust
Bartosz Nitka [Tue, 1 Dec 2015 15:42:35 +0000 (07:42 -0800)] 
Make the determinism tests more robust

The tests weren't explicit enough about comparing under different
unique allocation strategies. This led to some confusion on my part when
I started overriding flags in `testsuite/mk/test.mk`.
Includes a `.gitignore` rule.

Test Plan: harbormaster

Reviewers: austin, simonmar, bgamari

Reviewed By: bgamari

Subscribers: thomie

Differential Revision: https://phabricator.haskell.org/D1555

3 years agoFix warning about unused pattern variable
Gabor Greif [Tue, 1 Dec 2015 15:59:39 +0000 (16:59 +0100)] 
Fix warning about unused pattern variable

3 years agoRemove duplicated line
Gabor Greif [Wed, 28 Oct 2015 07:27:58 +0000 (08:27 +0100)] 
Remove duplicated line

3 years agoFix grammar and typo in TcTyDecls
Bartosz Nitka [Mon, 30 Nov 2015 16:44:27 +0000 (08:44 -0800)] 
Fix grammar and typo in TcTyDecls

Summary: It's just a docufix.

Test Plan: just a docufix

Reviewers: austin, bgamari

Reviewed By: bgamari

Subscribers: thomie

Differential Revision: https://phabricator.haskell.org/D1550

3 years agoBuild system: Add stage specific SRC_HC_(WARNING_)OPTS
Thomas Miedema [Mon, 30 Nov 2015 11:35:58 +0000 (12:35 +0100)] 
Build system: Add stage specific SRC_HC_(WARNING_)OPTS

* Add stage specific versions of SRC_HC_OPTS. These are currently only
  used for -Werror. The previous combination of GhcStage2HcOpts and
  GhcLibHcOpts didn't apply to utils/*.

* Add stage specific versions of SRC_HC_WARNING_OPTS. These will later be
  used for new warning supression flags that should not be passed to the
  bootstrap compiler.

* Move -Wall (and -Werror) related code back to mk/warnings.mk, where it
  was before 987d54274. Now all warning related code is nicely together.
  Include mk/warnings.mk after mk/custom-settings.mk to make this work.

Reviewed By: bgamari, hvr

Differential Revision: https://phabricator.haskell.org/D1536

3 years agoMkId: Typos in comments
Ömer Sinan Ağacan [Tue, 1 Dec 2015 02:07:24 +0000 (21:07 -0500)] 
MkId: Typos in comments