ghc.git
2 years agoMake InstalledUnitId be ONLY a FastString.
Edward Z. Yang [Thu, 6 Oct 2016 20:40:10 +0000 (13:40 -0700)] 
Make InstalledUnitId be ONLY a FastString.

It turns out that we don't really need to be able to
extract a ComponentId from UnitId, except in one case.
So compress UnitId into a single FastString.

The one case where we do need the ComponentId is when
we are compiling an instantiated version of a package;
we need the ComponentId to look up the indefinite
version of this package from the database.  So now we
just pass it in as an argument -this-component-id.

Also: ghc-pkg now no longer will unregister a package if
you register one with the same package name, if the
instantiations don't match.

Cabal submodule update which tracks the same data type
change.

Signed-off-by: Edward Z. Yang <ezyang@cs.stanford.edu>
2 years agoDistinguish between UnitId and InstalledUnitId.
Edward Z. Yang [Thu, 6 Oct 2016 07:17:15 +0000 (00:17 -0700)] 
Distinguish between UnitId and InstalledUnitId.

Signed-off-by: Edward Z. Yang <ezyang@cs.stanford.edu>
2 years agoThe Backpack patch.
Edward Z. Yang [Sat, 10 Oct 2015 19:01:14 +0000 (12:01 -0700)] 
The Backpack patch.

Summary:
This patch implements Backpack for GHC.  It's a big patch but I've tried quite
hard to keep things, by-in-large, self-contained.

The user facing specification for Backpack can be found at:

    https://github.com/ezyang/ghc-proposals/blob/backpack/proposals/0000-backpack.rst

A guide to the implementation can be found at:

    https://github.com/ezyang/ghc-proposals/blob/backpack-impl/proposals/0000-backpack-impl.rst

Has a submodule update for Cabal, as well as a submodule update
for filepath to handle more strict checking of cabal-version.

Signed-off-by: Edward Z. Yang <ezyang@cs.stanford.edu>
Test Plan: validate

Reviewers: simonpj, austin, simonmar, bgamari, goldfire

Subscribers: thomie, mpickering

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

2 years agoExclude Cabal PackageTests from gen_contents_index.
Edward Z. Yang [Thu, 6 Oct 2016 23:20:24 +0000 (16:20 -0700)] 
Exclude Cabal PackageTests from gen_contents_index.

Signed-off-by: Edward Z. Yang <ezyang@cs.stanford.edu>
2 years agoRemove reexports from ghc-boot, help bootstrap with GHC 8.
Edward Z. Yang [Sun, 2 Oct 2016 19:59:44 +0000 (12:59 -0700)] 
Remove reexports from ghc-boot, help bootstrap with GHC 8.

Signed-off-by: Edward Z. Yang <ezyang@cs.stanford.edu>
2 years agoNote about external interface changes.
Edward Z. Yang [Tue, 30 Aug 2016 04:20:58 +0000 (21:20 -0700)] 
Note about external interface changes.

Signed-off-by: Edward Z. Yang <ezyang@cs.stanford.edu>
2 years agoTc267, tests what happens if you forgot to knot-tie.
Edward Z. Yang [Thu, 25 Aug 2016 05:11:07 +0000 (22:11 -0700)] 
Tc267, tests what happens if you forgot to knot-tie.

Signed-off-by: Edward Z. Yang <ezyang@cs.stanford.edu>
2 years agoSimplify implementation of wWarningFlags
Matthew Pickering [Fri, 7 Oct 2016 14:19:56 +0000 (15:19 +0100)] 
Simplify implementation of wWarningFlags

2 years agoDon't suggest deprecated flags in error messages
Martin Ceresa [Fri, 7 Oct 2016 12:54:10 +0000 (13:54 +0100)] 
Don't suggest deprecated flags in error messages

When looking up flags, we make sure to lookup the non-deprecated flags first
by ordering the list of flags.

Reviewers: bgamari, austin, mpickering

Reviewed By: mpickering

Subscribers: mpickering, thomie

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

GHC Trac Issues: #12574

2 years agoFix memory leak from #12664
Bartosz Nitka [Thu, 6 Oct 2016 12:40:24 +0000 (05:40 -0700)] 
Fix memory leak from #12664

This fixes the leak with `setProgArgv`. The problem was
that `setProgArgv` would not free the objects pointed
to by `prog_argc`, `prog_argv` when the globals were
changed resulting in a leak.

The only strictly necessary change is in `rts/RtsFlags.c`, but
the code in `System.Environment` was a bit confusing and not
exception safe, so I refactored it.

Test Plan: ./validate

Reviewers: simonmar, ezyang, austin, hvr, bgamari, erikd

Subscribers: thomie

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

GHC Trac Issues: #12664

2 years agoAdd compact to packages so it gets cleaned on make clean.
Edward Z. Yang [Thu, 6 Oct 2016 23:48:56 +0000 (16:48 -0700)] 
Add compact to packages so it gets cleaned on make clean.

Signed-off-by: Edward Z. Yang <ezyang@cs.stanford.edu>
2 years agoRemove dead code “mkHsConApp”
Joachim Breitner [Wed, 5 Oct 2016 20:14:59 +0000 (16:14 -0400)] 
Remove dead code “mkHsConApp”

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

2 years agoRegAlloc: Make some pattern matched complete
Joachim Breitner [Thu, 6 Oct 2016 03:37:03 +0000 (23:37 -0400)] 
RegAlloc: Make some pattern matched complete

these actually are complete, but due to the use of pattern guards, the
compiler does not see that. Refactor the code that it does.

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

2 years agoRefactor TcDeriv and TcGenDeriv
Ryan Scott [Thu, 6 Oct 2016 13:14:49 +0000 (09:14 -0400)] 
Refactor TcDeriv and TcGenDeriv

Summary:
Keeping a promise I made to Simon to clean up these modules.

This change splits up the massive `TcDeriv` and `TcGenDeriv` modules into
somewhat more manageable pieces. The new modules are:

* `TcGenFunctor`: This contains the deriving machinery for `Functor`,
  `Foldable`, and `Traversable` (which all use the same underlying algorithm).
* `TcDerivInfer`: This is the new home for `inferConstraints`,
  `simplifyInstanceContexts`, and related functions, whose role is to come up
  with the derived instance context and subsequently simplify it.
* `TcDerivUtils`: This is a grab-bag module that contains several
  error-checking utilities originally in `TcDeriv`, as well as some functions
  that `TcDeriv` and `TcDerivInfer` both need.

The end result is that `TcDeriv` is now less than 1,600 SLOC (originally 2,686
SLOC), and `TcGenDeriv` is now about 2,000 SLOC (originally 2,964).

In addition, this also implements a couple of tiny refactorings:

* I transformed `type Condition = (DynFlags, TyCon) -> Validity` into
  `type Condition = DynFlags -> TyCon -> Validity`
* I killed the `DerivSpecGeneric` constructor for `DerivSpecMechanism`, and
  merged its functionality into `DerivSpecStock`. In addition,
  `hasStockDeriving` now contains key-value pairs for `Generic` and `Generic1`,
  so they're no longer treated as an awkward special case in `TcDeriv`.

Test Plan: ./validate

Reviewers: simonpj, austin, bgamari

Reviewed By: simonpj

Subscribers: thomie, mpickering

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

2 years agoRemove unused T12124.srderr
Ryan Scott [Thu, 6 Oct 2016 13:12:05 +0000 (09:12 -0400)] 
Remove unused T12124.srderr

This was (accidentally?) introduced in 465c6c5d15f8fb54afb78408f3a79e75e74d2cd4

2 years agoImprove error handling in TcRnMonad
Simon Peyton Jones [Wed, 5 Oct 2016 21:00:02 +0000 (22:00 +0100)] 
Improve error handling in TcRnMonad

See Note [Constraints and errors] in TcRnMonad.  This
patch fixes Trac #12124 in quite a neat way.

2 years agoBring Note in TcDeriv up to date
Ryan Scott [Wed, 5 Oct 2016 20:13:29 +0000 (16:13 -0400)] 
Bring Note in TcDeriv up to date

Comments only.

[ci skip]

2 years agoMark zipWithAndUnzipM as INLINABLE rather than INLINE
Matthew Pickering [Tue, 4 Oct 2016 21:36:55 +0000 (22:36 +0100)] 
Mark zipWithAndUnzipM as INLINABLE rather than INLINE

It is a self-recursive function and hence a loop-breaker.

2 years agogenapply: update source file in autogenerated text
Sergei Trofimovich [Tue, 4 Oct 2016 20:13:51 +0000 (21:13 +0100)] 
genapply: update source file in autogenerated text

Signed-off-by: Sergei Trofimovich <siarheit@google.com>
2 years agorunghc: Fix import of System.Process on Windows
Ben Gamari [Sun, 2 Oct 2016 23:40:56 +0000 (19:40 -0400)] 
runghc: Fix import of System.Process on Windows

This apparently should have been an import of rawSystem instead of
runProcess. Oops.

Fixes D2538.

Test Plan: Validate on Linux and Windows.

Reviewers: austin, snowleopard

Reviewed By: snowleopard

Subscribers: thomie

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

2 years agoUpdate Cabal submodule to latest version.
Edward Z. Yang [Fri, 30 Sep 2016 23:50:52 +0000 (16:50 -0700)] 
Update Cabal submodule to latest version.

Summary:
Note that Cabal needs one more bugfix which is in PR to
fix GHC bootstrapping. But the rest of the patch is
ready for review.

Needs a filepath submodule update because cabal check
became more strict.

This patch handles the abstract-ification of Version and
PackageName.

Signed-off-by: Edward Z. Yang <ezyang@cs.stanford.edu>
Test Plan: validate

Reviewers: bgamari, austin

Subscribers: thomie

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

2 years agotestsuite: Mark test for #12355 as unbroken on Darwin.
Ben Gamari [Sun, 2 Oct 2016 17:47:03 +0000 (13:47 -0400)] 
testsuite: Mark test for #12355 as unbroken on Darwin.

Somehow this testcase works on Darwin but not on Linux. This deserves
further investigation.

2 years agoDynFlags: Fix absolute import path to generated header
Ben Gamari [Sun, 2 Oct 2016 17:46:12 +0000 (13:46 -0400)] 
DynFlags: Fix absolute import path to generated header

Test Plan: Validate

Reviewers: austin, snowleopard

Reviewed By: snowleopard

Subscribers: thomie

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

GHC Trac Issues: #8040.

2 years agoIgnore output from derefnull and divbyzero on Darwin
Ben Gamari [Sun, 2 Oct 2016 16:01:21 +0000 (12:01 -0400)] 
Ignore output from derefnull and divbyzero on Darwin

The output contains the pid and executable path of the bash process
which spawned the failing process. It doesn't seem worth the effort to
cleanse this output; just ignore it.

2 years agoMark #6132 as broken on OS X
Ben Gamari [Sun, 2 Oct 2016 15:53:54 +0000 (11:53 -0400)] 
Mark #6132 as broken on OS X

It currently fails with,

    =====> T6132(normal) 1 of 1 [0, 0, 0]
    cd "./runghc/T6132.run" && "/Users/bgamari/ghc/inplace/test
    spaces/ghc-stage2" -c T6132.hs -dcore-lint -dcmm-lint
    -no-user-package-db -rtsopts -fno-warn-missed-specialisations
    -fshow-warning-groups -dno-debug-output
    Compile failed (exit code 1) errors were:

    T6132.hs:1:2: error: parse error on input ‘#!/’

    *** unexpected failure for T6132(normal)

2 years agorunghc: use executeFile to run ghc process on POSIX
Michael Snoyman [Sun, 2 Oct 2016 01:24:05 +0000 (21:24 -0400)] 
runghc: use executeFile to run ghc process on POSIX

This means that, on POSIX systems, there will be only one ghc process
used for running scripts, as opposed to the current situation of a
runghc process and a ghc process. Beyond minor performance benefits of
not having an extra fork and resident process, the more important impact
of this is automatically getting proper signal handling. I noticed this
problem myself when running runghc as PID1 inside a Docker container.

I attempted to create a shim library for executeFile that would work for
both POSIX and Windows, but unfortunately I ran into issues with exit
codes being propagated correctly (see
https://github.com/fpco/replace-process/issues/2). Therefore, this patch
leaves the Windows behavior unchanged. Given that signals are a POSIX
issue, this isn't too bad a trade-off. If someone has suggestions for
better Windows _exec support, please let me know.

Reviewers: erikd, austin, bgamari

Reviewed By: bgamari

Subscribers: Phyx, thomie

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

2 years agovalidate: Add --build-only
Ben Gamari [Sun, 2 Oct 2016 00:11:03 +0000 (20:11 -0400)] 
validate: Add --build-only

This will allow us to split up Harbormaster output for the build and
test stages of validation.

Test Plan: `./validate --build-only && ./validate --testsuite-only`

Reviewers: thomie, hvr, austin

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

2 years agoDisallow standalone deriving declarations involving unboxed tuples or sums
Ryan Scott [Sat, 1 Oct 2016 21:58:44 +0000 (17:58 -0400)] 
Disallow standalone deriving declarations involving unboxed tuples or sums

There was an awful leak where GHC permitted standalone `deriving`
declarations to create instances for unboxed sum or tuple types. This
fortifies the checks that GHC performs to catch this scenario and give
an appropriate error message.

Fixes #11509.

Test Plan: ./validate

Reviewers: goldfire, austin, bgamari

Reviewed By: bgamari

Subscribers: thomie

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

GHC Trac Issues: #11509

2 years agoEliminate some unsafeCoerce#s with deriving strategies
Ryan Scott [Sat, 1 Oct 2016 21:58:27 +0000 (17:58 -0400)] 
Eliminate some unsafeCoerce#s with deriving strategies

Currently, `Foreign.C.Types`, `Foreign.Ptr`, and `System.Posix.Types`
define `Read` and `Show` instances for the newtypes in those modules by
using `unsafeCoerce#`. We can clean up this hack by using the `newtype`
deriving strategy.

Reviewers: hvr, austin, bgamari

Reviewed By: bgamari

Subscribers: thomie

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

2 years agoTurn `__GLASGOW_HASKELL_LLVM__` into an integer again
Nicolas Trangez [Sat, 1 Oct 2016 21:58:11 +0000 (17:58 -0400)] 
Turn `__GLASGOW_HASKELL_LLVM__` into an integer again

In GHC < 8.0.1, the value of `__GLASGOW_HASKELL_LLVM__`, exposed
through the preprocessor when compiled with `-fllvm`, was an integer
value, encoded according to some rules specified in the user guide.

Due to an oversight, in GHC 8.0.1 the value of this define became a
tuple, exposed as e.g. `(3, 7)`. This was an unintended regression.

This patch turns the value of the `__GLASGOW_HASKELL_LLVM__` definition
into a single integer again, but changes the formatting of said number
slightly. Before, any LLVM version where the major or minor component >=
10 would cause ambiguous values for `__GLASGOW_HASKELL_LLVM__`. With
this patch, the value is in line with `__GLASGOW_HASKELL__`, adding a
padding `0` in-between major and minor component if applicable (we
assume no minors >= 100 will ever exist).

The documentation in the user guide is updated accordingly, and a
reference is made in the 8.0.2 release notes.

Test Plan: validate

Reviewers: bgamari, erikd

Reviewed By: bgamari

Subscribers: thomie

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

GHC Trac Issues: #12628

2 years agoghc-pkg: Allow unregistering multiple packages in one call
Niklas Hambüchen [Sat, 1 Oct 2016 21:57:32 +0000 (17:57 -0400)] 
ghc-pkg: Allow unregistering multiple packages in one call

Reviewers: austin, bgamari

Reviewed By: bgamari

Subscribers: thomie

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

GHC Trac Issues: #12637

2 years agoDo not warn about unused underscore-prefixed fields (fixes Trac #12609)
Adam Gundry [Sat, 1 Oct 2016 21:56:58 +0000 (17:56 -0400)] 
Do not warn about unused underscore-prefixed fields (fixes Trac #12609)

When DuplicateRecordFields is enabled, the mangling of selector names
was causing them to be reported as unused even if prefixed by an
underscore. This corrects the OccName used by the check.

Test Plan: New test overloadedrecflds/should_compile/T12609

Reviewers: austin, bgamari

Reviewed By: bgamari

Subscribers: thomie

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

GHC Trac Issues: #12609

2 years agoPPC/CodeGen: fix lwa instruction generation
Peter Trommler [Sat, 1 Oct 2016 21:56:31 +0000 (17:56 -0400)] 
PPC/CodeGen: fix lwa instruction generation

Opcode lwa is a 64-bit opcode and allows a DS-form only.  This patch
generates lwa opcodes only when the offset is a multiple of 4.

Fixes #12621

Test Plan: validate

Reviewers: erikd, hvr, simonmar, austin, bgamari

Reviewed By: bgamari

Subscribers: thomie

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

GHC Trac Issues: #12621

2 years agoDon't warn about name shadowing when renaming the patten in a PatSyn decl
Matthew Pickering [Sat, 1 Oct 2016 21:55:26 +0000 (17:55 -0400)] 
Don't warn about name shadowing when renaming the patten in a PatSyn decl

Previously the renamer assumed that *any* time we renamed a pattern, the
pattern was introducing new binders. This isn't true in pattern synonym
declarations where the pattern is used as part of a definition.

We add a special case to not warn in this situation.

Reviewers: simonpj, austin, bgamari

Reviewed By: simonpj

Subscribers: simonpj, thomie

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

GHC Trac Issues: #12615

2 years agoFix interaction of record pattern synonyms and record wildcards
Matthew Pickering [Sat, 1 Oct 2016 21:55:04 +0000 (17:55 -0400)] 
Fix interaction of record pattern synonyms and record wildcards

We were missing an appropiate *ConLike lookup in the case when
the pattern synonym was defined in a different module.

Reviewers: austin, bgamari, simonpj

Reviewed By: simonpj

Subscribers: simonpj, thomie

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

GHC Trac Issues: #11987

2 years agoAdd NUMA support for Windows
Tamar Christina [Sun, 25 Sep 2016 19:00:31 +0000 (20:00 +0100)] 
Add NUMA support for Windows

Summary:
NOTE: I have been able to do simple testing on emulated NUMA nodes.
           Real hardware would be needed for a proper test.

D2199 Added NUMA support for Linux, I have just filled in the missing pieces following
the description of the Linux APIs.

Test Plan:
Use `bcdedit.exe /set groupsize 2` to modify the kernel again (Similar to D2533).

This generates some NUMA nodes:

```
Logical Processor to NUMA Node Map:
NUMA Node 0:
**
--
NUMA Node 1:
--
**

Approximate Cross-NUMA Node Access Cost (relative to fastest):
     00  01
00: 1.1 1.1
01: 1.0 1.0
```

run ` ../test-numa.exe +RTS --numa -RTS`

and check PerfMon for NUMA allocations.

Reviewers: simonmar, erikd, bgamari, austin

Reviewed By: simonmar

Subscribers: thomie, #ghc_windows_task_force

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

GHC Trac Issues: #12602

2 years agoUse check stacking on Windows.
Tamar Christina [Sat, 17 Sep 2016 21:25:00 +0000 (22:25 +0100)] 
Use check stacking on Windows.

Summary:
#8870 added as a temporary work around a much higher initial reserve
and committed stack space of 2mb. This is causing problems with other windows applications.

The hack was supposed to be temporary untill we could emit `__chkstk` instructions.
But GCC can emit stack checks automatically for us if `-fstack-check` is passed.

This will then emit calls to `___chkstk_ms` before stack allocations.

```
  633de0:       48 83 e0 f0             and    $0xfffffffffffffff0,%rax
  633de4:       e8 07 0c 0d 00          callq  7049f0 <___chkstk_ms>
  633de9:       48 29 c4                sub    %rax,%rsp
```

The hack is now no longer needed.

Test Plan: ./validate

Reviewers: austin, erikd, awson, bgamari

Reviewed By: bgamari

Subscribers: thomie, #ghc_windows_task_force

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

GHC Trac Issues: #12186

2 years agoMark T11978a as broken due to #12019
Ben Gamari [Sat, 1 Oct 2016 21:02:49 +0000 (17:02 -0400)] 
Mark T11978a as broken due to #12019

Test Plan: `validate --slow`

Reviewers: austin

Subscribers: thomie

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

GHC Trac Issues: #12019

2 years agoMove -dno-debug-output to the end of the test flags
Matthew Pickering [Sat, 1 Oct 2016 04:28:43 +0000 (00:28 -0400)] 
Move -dno-debug-output to the end of the test flags

It is often convenient to copy the test invocation and remove this flag
in order to see compiler traces. Moving it to the end makes it easier to
remove.

Remove trailing whitespace

Reviewers: austin, thomie, bgamari

Reviewed By: bgamari

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

2 years agoRecognise US spelling for specialisation flags.
Tim McGilchrist [Sat, 1 Oct 2016 04:28:26 +0000 (00:28 -0400)] 
Recognise US spelling for specialisation flags.

The user guide says that we allow the user to use `specialise` or
`specialize` interchangeably but this wasn't the case for the relevant
flags. This patch adds aliases for the flags which control
specialisation.

Reviewers: erikd, austin, mpickering, bgamari

Reviewed By: mpickering, bgamari

Subscribers: mpickering, thomie

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

GHC Trac Issues: #12575

2 years agoSupport more than 64 logical processors on Windows
Tamar Christina [Sat, 1 Oct 2016 04:26:52 +0000 (00:26 -0400)] 
Support more than 64 logical processors on Windows

Windows support for more than 64 logical processors are implemented
using processor groups.

Essentially what it's doing is keeping the existing maximum of 64
processors and keeping the affinity mask a 64 bit value, but adds an
hierarchy above that.

This support was added to Windows 7 and so we need to at runtime detect
if the APIs are still there due to our minimum supported version being
Windows Vista.

The Maximum number of groups supported at this time is 4, so 256 logical
cores.  The group indices are 0 based. One thread can have affinity with
multiple groups.

See
https://msdn.microsoft.com/en-us/library/windows/desktop/ms684251.aspx
and particularly helpful is the whitepaper: 'Supporting Systems that
have more than 64 processors' at
https://msdn.microsoft.com/en-us/library/windows/hardware/dn653313.aspx

Processor groups are not guaranteed to be uniformly distributed nor
guaranteed to be filled before a next group is needed. The OS will
assign processors to groups based on physical proximity and will never
partially assign cores from one physical cpu to more than one group. If
one has two 48 core CPUs then you'd end up with two groups of 48 logical
cpus. Now add a 3rd CPU with 10 cores and the group it is assigned to
depends where the socket is on the board.

Test Plan:
./validate or make test -c . in the rts test folder.

This tests for regressions, to test this particular functionality
itself:

   <program> +RTS -N -qa -RTS

Test is detailed in description.

Reviewers: bgamari, simonmar, austin, erikd

Reviewed By: simonmar

Subscribers: thomie, #ghc_windows_task_force

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

GHC Trac Issues: #11054

2 years agoGHCi: Don't remove shadowed bindings from typechecker scope.
mniip [Sat, 1 Oct 2016 04:26:04 +0000 (00:26 -0400)] 
GHCi: Don't remove shadowed bindings from typechecker scope.

The shadowed out bindings are accessible via qualified names like
Ghci1.foo.  Since they are accessable in the renamer the typechecker
should be able to see them too.  As a consequence they show up in :show
bindings.

This fixes T11547

Test Plan:
Fixed current tests to accomodate to new stuff in :show bindings
Added a test that verifies that the typechecker doesn't crash

Reviewers: austin, bgamari, simonpj

Reviewed By: simonpj

Subscribers: simonpj, thomie

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

GHC Trac Issues: #11547

2 years agoCodeGen X86: fix unsafe foreign calls wrt inlining
Sylvain HENRY [Sat, 1 Oct 2016 04:25:49 +0000 (00:25 -0400)] 
CodeGen X86: fix unsafe foreign calls wrt inlining

Foreign calls (unsafe and safe) interact badly with inlining and
register passing ABIs (see #11792 and #12614):
the inlined code to compute a parameter of the call may overwrite a
register already set to pass a preceding parameter.

With this patch, we compute all parameters which are not simple
expressions before assigning them to fixed registers required by the
ABI.

Test Plan:
   - Add test (test both reg and stack parameters)
   - Validate

Reviewers: osa1, bgamari, austin, simonmar

Reviewed By: simonmar

Subscribers: thomie

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

GHC Trac Issues: #11792, #12614

2 years agoImplement deriving strategies
Ryan Scott [Sat, 1 Oct 2016 00:15:25 +0000 (20:15 -0400)] 
Implement deriving strategies

Allows users to explicitly request which approach to `deriving` to use
via keywords, e.g.,

```
newtype Foo = Foo Bar
  deriving Eq
  deriving stock    Ord
  deriving newtype Show
```

Fixes #10598. Updates haddock submodule.

Test Plan: ./validate

Reviewers: hvr, kosmikus, goldfire, alanz, bgamari, simonpj, austin,
erikd, simonmar

Reviewed By: alanz, bgamari, simonpj

Subscribers: thomie, mpickering, oerjan

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

GHC Trac Issues: #10598

2 years agoDocument Safe Haskell restrictions on Generic instances
Ryan Scott [Fri, 30 Sep 2016 19:52:02 +0000 (15:52 -0400)] 
Document Safe Haskell restrictions on Generic instances

Commit 578fbeca31dd3d755e24e910c3a7327f92bc4ee3 disallowed hand-written
`Generic` instances in Safe Haskell, but this wasn't documented anywhere.

2 years agoCorrect spelling in note references
Gabor Greif [Fri, 30 Sep 2016 14:37:53 +0000 (16:37 +0200)] 
Correct spelling in note references

2 years agoMake tcrun042 fail
Simon Peyton Jones [Fri, 30 Sep 2016 13:33:19 +0000 (14:33 +0100)] 
Make tcrun042 fail

This test uses wholesale impredicative polymorphism, and now fails.
That's ok.

2 years agoAdd missing stderr file
Simon Peyton Jones [Fri, 30 Sep 2016 12:32:51 +0000 (13:32 +0100)] 
Add missing stderr file

..accidentally omitted from previous commit.

2 years agoComments only
Simon Peyton Jones [Tue, 27 Sep 2016 16:16:57 +0000 (17:16 +0100)] 
Comments only

2 years agoFix impredicativity (again)
Simon Peyton Jones [Sun, 25 Sep 2016 14:50:18 +0000 (15:50 +0100)] 
Fix impredicativity (again)

This patch fixes Trac #12616.

Dignosis.  In TcUnify.tc_sub_type_ds we were going to some trouble to
support co- and contra-variance even for impredicative types.  With
-XImpredicativeTYpes, this allowed a unification variable to be
unified with a polytype (probably wrongly) and that caused later
trouble in the constraint solver, where -XImpredicativeTypes was /not/
on.  In effect, -XImpredicativeTypes can't be switched on locally.

Why did we want ImpredicativeTypes locally?  Because the program
generated by GND for a higher-rank method involved impredicative
instantation of 'coerce':
      op = coerce op   -- where op has a higher rank type
See Note [Newtype-deriving instances] in TcGenDeriv.

Cure.

1.  It is ghastly to rely on ImpredicativeTypes (a 100% flaky
    feature) to instantiate coerce polymorphically.  Happily we
    now have Visible Type Application, so I've used that instead
    which should be solid and reliable.

2.  I deleted the code in tc_sub_type_ds that allows the constraint
    solver to "look through" a unification variable to find a
    polytype.  That used to be essential in the days of ReturnTv,
    but it's utterly unreliable and should be consigned to the dustbin
    of history.  (We have ExpType now for the essential uses.)

Tests involving ImpredicativeTypes are affected, but I'm not worried
about them... it's advertised as a feature you can't rely on, and
I want to reform it outright.

2 years agoAdd Outputable Report in TcErrors
Simon Peyton Jones [Sun, 25 Sep 2016 14:48:29 +0000 (15:48 +0100)] 
Add Outputable Report in TcErrors

...just for debug output

2 years agoFix a bug in occurs checking
Simon Peyton Jones [Thu, 22 Sep 2016 21:18:22 +0000 (22:18 +0100)] 
Fix a bug in occurs checking

1. Trac #12593 exposed a long-standing bug in the occurs
   checking machinery.  When unifying two type variables
          a ~ b
   where a /= b, we were assuming that there could be
   no occurs-check error.  But there can: 'a' can occur
   in b's kind!  When the RHS was a non-tyvar we used
   occurCheckExpand, which /did/ look in kinds, but not
   when the RHS was a tyvar.

   This bug has been lurking ever since TypeInType, maybe
   longer.  And it was present both in TcUnify (the on-the-fly
   unifier), and in TcInteract.

   I ended up refactoring both so that the tyvar/tyvar
   path naturally goes through the same occurs-check as
   non-tyvar rhss.  It's simpler and more robust now.

   One good thing is that both unifiers now share
       TcType.swapOverVars
       TcType.canSolveByUnification
   previously they had different logic for the same goals

2. Fixing this bug exposed another!  In T11635 we end
   up unifying
   (alpha :: forall k. k->*) ~ (beta :: forall k. k->*)
   Now that the occurs check is done for tyvars too, we
   look inside beta's kind.  And then reject the program
   becuase of the forall inside there.  But in fact that
   forall is fine -- it does not count as impredicative
   polymoprhism.   See Note [Checking for foralls]
   in TcType.

3. All this fuss around occurrence checking forced me
   to look at TcUnify.checkTauTvUpdate
          and TcType.occurCheckExpand
   There's a lot of duplication there, and I managed
   to elminate quite a bit of it. For example,
   checkTauTvUpdate called a local 'defer_me'; and then
   called occurCheckExpand, which then used a very
   similar 'fast_check'.

   Things are better, but there is more to do.

2 years agoFix desugaring of pattern bindings (again)
Simon Peyton Jones [Fri, 16 Sep 2016 21:33:20 +0000 (22:33 +0100)] 
Fix desugaring of pattern bindings (again)

This patch fixes Trac #12595.  The problem was with a
pattern binding like
     !x = e
For a start it's silly to match that pattern and build
a unit tuple (the General Case of mkSelectorBinds); but
that's what was happening because the bang fell through
to the general case.  But for a variable pattern building
any auxiliary bindings is stupid.  So the patch
introduces a new case in mkSelectorBinds for variable
patterns.

Then it turned out that if 'e' was a plain variable, and
moreover was imported GlobalId, then matchSinglePat made
it a /bound/ variable, which should never happen.  That
ultimately caused a linker error, but the original bug
was much earlier.

2 years agoA bit of tracing about flattening
Simon Peyton Jones [Sat, 24 Sep 2016 03:42:18 +0000 (04:42 +0100)] 
A bit of tracing about flattening

2 years agoCheck.hs: Use actual import lists instead of comments
Ömer Sinan Ağacan [Thu, 29 Sep 2016 15:03:24 +0000 (11:03 -0400)] 
Check.hs: Use actual import lists instead of comments

2 years agoTest Trac #12634
Simon Peyton Jones [Thu, 29 Sep 2016 08:23:11 +0000 (09:23 +0100)] 
Test Trac #12634

2 years agoComments and manual only: spelling
Gabor Greif [Wed, 28 Sep 2016 13:28:38 +0000 (15:28 +0200)] 
Comments and manual only: spelling

2 years agoTH: Use atomicModifyIORef' for fresh names
Erik de Castro Lopo [Tue, 27 Sep 2016 21:37:33 +0000 (07:37 +1000)] 
TH: Use atomicModifyIORef' for fresh names

This prevents the possibility of race conditions when creating fresh
names.

Test Plan: validate

Reviewers: goldfire, austin, bgamari

Reviewed By: bgamari

Subscribers: thomie

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

2 years agoFix layout of MultiWayIf expressions (#10807)
Ömer Sinan Ağacan [Mon, 26 Sep 2016 21:09:01 +0000 (17:09 -0400)] 
Fix layout of MultiWayIf expressions (#10807)

With this patch we stop generating virtual semicolons in MultiWayIf
guards. Fixes #10807.

Test Plan:

Reviewers: simonmar, austin, bgamari

Reviewed By: simonmar

Subscribers: mpickering, thomie

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

GHC Trac Issues: #10807

2 years agoRemove #ifdef with never fulfilled condition
Joachim Breitner [Mon, 26 Sep 2016 17:49:42 +0000 (13:49 -0400)] 
Remove #ifdef with never fulfilled condition

__GLASGOW_HASKELL is a typo (should be __GLASGOW_HASKELL__), obviously,
but nobody noticed.

2 years agoPrint foralls in user format
Simon Peyton Jones [Mon, 26 Sep 2016 07:37:47 +0000 (08:37 +0100)] 
Print foralls in user format

This fixes Trac #12597: in RnNames.warnMissingSignatures,
use pprSigmaType not pprType

2 years agoPattern Synonyms documentation update
Matthew Pickering [Sat, 24 Sep 2016 15:28:02 +0000 (16:28 +0100)] 
Pattern Synonyms documentation update

2 years agoUpdate test output
Joachim Breitner [Sat, 24 Sep 2016 01:20:18 +0000 (10:20 +0900)] 
Update test output

2 years agoReplace INLINEABLE by INLINABLE (#12613)
Joachim Breitner [Sat, 24 Sep 2016 00:16:59 +0000 (09:16 +0900)] 
Replace INLINEABLE by INLINABLE (#12613)

as the latter is the official, correct spelling, and the former just a
misspelling accepted by GHC.

Also document in the user’s guide that the alternative spelling is
accepted

This commit was brough to you by HIW 2016.

2 years agoMark mapUnionFV as INLINABLE rather than INLINE
Matthew Pickering [Fri, 23 Sep 2016 22:25:43 +0000 (23:25 +0100)] 
Mark mapUnionFV as INLINABLE rather than INLINE

It is a self-recursive function so will always be the loop-breaker
and hence never able to be inlined. It is dubious whether the INLINABLE
pragma will ever help as it is not a very polymorphic function
but some specialisation could occur.

2 years agoFix #12442.
Richard Eisenberg [Wed, 27 Jul 2016 20:41:50 +0000 (16:41 -0400)] 
Fix #12442.

The problem is described in the ticket.

This patch adds new variants of the access points to the pure
unifier that allow unification of types only when the caller
wants this behavior. (The unifier used to also unify kinds.)
This behavior is appropriate when the kinds are either already
known to be the same, or the list of types provided are a
list of well-typed arguments to some type constructor. In the
latter case, unifying earlier types in the list will unify the
kinds of any later (dependent) types.

At use sites, I went through and chose the unification function
according to the criteria above.

This patch includes some modest performance improvements as we
are now doing less work.

2 years agoImproved documentation for Foreign.Concurrent (#12547)
Simon Marlow [Fri, 16 Sep 2016 13:48:57 +0000 (14:48 +0100)] 
Improved documentation for Foreign.Concurrent (#12547)

We had better docs for the underlying functions in GHC.ForeignPtr, but
this wasn't propagated to the re-exported versions in
Foreign.Concurrent.

2 years agoExpose hs_exit_(rtsFalse) as hs_exit_nowait()
Simon Marlow [Fri, 16 Sep 2016 10:04:44 +0000 (11:04 +0100)] 
Expose hs_exit_(rtsFalse) as hs_exit_nowait()

Summary: And document it.  See the docmentation for the reason I want this.

Test Plan: It's an existing API, just exposing it.

Reviewers: bgamari, niteria, austin, erikd

Subscribers: thomie

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

2 years agoshutdownHaskellAndExit: just do a normal hs_exit() (#5402)
Simon Marlow [Thu, 15 Sep 2016 14:53:00 +0000 (15:53 +0100)] 
shutdownHaskellAndExit: just do a normal hs_exit() (#5402)

If we want to keep the RTS alive a bit longer by having another
hs_init()/hs_exit() pair in a library that will destruct itself after
main() has exited, then the forced shutdown here thwarts that.

I think we just "fixed" #5402 in the wrong way before, this should be
better.

2 years agoFix failing test T12504
Tamar Christina [Wed, 21 Sep 2016 00:51:47 +0000 (01:51 +0100)] 
Fix failing test T12504

Summary:
Test T12504 does it's checking in the Makefile using grep but still specified an stdout.
the stdout has the old output and would always fail.

Since the stdout isn't needed, let's not check it.

Test Plan: make test TEST=T12504

Reviewers: bgamari, austin, erikd, rcook

Reviewed By: rcook

Subscribers: thomie, #ghc_windows_task_force

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

GHC Trac Issues: #12504

2 years agoBump Haddock submodule to fix rendering of class methods
Matthew Pickering [Tue, 20 Sep 2016 18:48:01 +0000 (19:48 +0100)] 
Bump Haddock submodule to fix rendering of class methods

Fixes #12519

2 years agoCodegen for case: Remove redundant void id checks
Ömer Sinan Ağacan [Tue, 20 Sep 2016 04:19:15 +0000 (00:19 -0400)] 
Codegen for case: Remove redundant void id checks

New unarise (714bebf) eliminates void binders in patterns already, so no
need to eliminate them here. I leave assertions to make sure this is the
case.

Assertion failure -> bug in unarise

Reviewers: bgamari, simonpj, austin, simonmar, hvr

Reviewed By: simonpj

Subscribers: thomie

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

2 years agoRemove directories from include paths
Ben Gamari [Fri, 16 Sep 2016 15:49:24 +0000 (11:49 -0400)] 
Remove directories from include paths

Previously this was a relative path which worked in the GHC tree, but
failed elsewhere. This caused trouble for out-of-tree users as well as
Hadrian, which wants to move build artifacts out of the working
directory. Fixes #8040.

Test Plan: Validate

Reviewers: thomie, austin, snowleopard, hvr

Reviewed By: snowleopard, hvr

Subscribers: thomie

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

GHC Trac Issues: #8040

2 years agoAdd test case for #7611
Joachim Breitner [Thu, 15 Sep 2016 21:06:16 +0000 (17:06 -0400)] 
Add test case for #7611

basically using the machinery from the test case of #2110.

2 years agoComments in TH.Syntax (Trac #12596)
Simon Peyton Jones [Wed, 14 Sep 2016 07:38:33 +0000 (08:38 +0100)] 
Comments in TH.Syntax (Trac #12596)

See Note [Data for non-algebraic types]

2 years agoComments only
Simon Peyton Jones [Thu, 15 Sep 2016 16:07:12 +0000 (17:07 +0100)] 
Comments only

2 years agoUnify CallStack handling in ghc
Ben Gamari [Mon, 12 Sep 2016 18:54:30 +0000 (14:54 -0400)] 
Unify CallStack handling in ghc

Here we introduce compatibility wrappers for HasCallStack constraints.
This is necessary as we must support GHC 7.10.1 which lacks sane call
stack support. We also introduce another constraint synonym,
HasDebugCallStack, which only provides a call stack when DEBUG is set.

2 years agotryGrabCapability should be using TRY_ACQUIRE_LOCK
Simon Marlow [Thu, 15 Sep 2016 10:34:05 +0000 (11:34 +0100)] 
tryGrabCapability should be using TRY_ACQUIRE_LOCK

2 years agoFix codegen bug in PIC version of genSwitch (#12433)
Simon Marlow [Thu, 15 Sep 2016 09:11:34 +0000 (10:11 +0100)] 
Fix codegen bug in PIC version of genSwitch (#12433)

Summary:
* getNonClobberedReg instead of getSomeReg, because the reg needs to
  survive across t_code
* Use a new reg for the table offset calculation instead of clobbering
  the reg returned by expr (this was the bug affecting #12433)

Test Plan: New unit test; validate

Reviewers: rwbarton, bgamari, austin, erikd

Subscribers: thomie

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

GHC Trac Issues: #12433

2 years agoComments and white space
Simon Peyton Jones [Mon, 5 Sep 2016 15:52:35 +0000 (16:52 +0100)] 
Comments and white space

2 years agoComments and white space only
Simon Peyton Jones [Mon, 5 Sep 2016 08:28:19 +0000 (09:28 +0100)] 
Comments and white space only

2 years agoComments and white space only
Simon Peyton Jones [Mon, 29 Aug 2016 08:04:24 +0000 (09:04 +0100)] 
Comments and white space only

2 years agoTestsuite wibbles, to the same files
Simon Peyton Jones [Mon, 12 Sep 2016 23:00:21 +0000 (00:00 +0100)] 
Testsuite wibbles, to the same files

Sigh.  I added some comments to the source files,
and failed to revalidate.  But the comments change the
line number in the error messages.  Doh.

2 years agoTest wibbles for commit 03541cba
Simon Peyton Jones [Mon, 12 Sep 2016 21:21:15 +0000 (22:21 +0100)] 
Test wibbles for commit 03541cba

I must have failed to validate properly, sorry.
These testsuite wibbles belong with

  commit 03541cbae50f0d1cdf99120ab88698f29a278159
  Author: Simon Peyton Jones <simonpj@microsoft.com>
  Date:   Fri Sep 9 17:42:42 2016 +0100

      Be less picky about reporing inaccessible code

2 years agorts: Disable -hb with multiple capabilities
Ben Gamari [Mon, 12 Sep 2016 12:44:24 +0000 (08:44 -0400)] 
rts: Disable -hb with multiple capabilities

Biographical profiling is not thread-safe as documented in #12019. Throw
an error when it is used in this way.

Test Plan: Validate

Reviewers: simonmar, austin, erikd

Reviewed By: erikd

Subscribers: thomie

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

GHC Trac Issues: #12019

2 years agousers_guide: TH now partially supports typed holes
Ben Gamari [Thu, 8 Sep 2016 15:34:11 +0000 (11:34 -0400)] 
users_guide: TH now partially supports typed holes

As requested in #10267. However we still lack support in typed splices.
See #10945 and #10946.

2 years agousers_guide: #8761 is now fixed
Ben Gamari [Thu, 8 Sep 2016 15:31:44 +0000 (11:31 -0400)] 
users_guide: #8761 is now fixed

Remove a leftover note in the users guide claiming that TH doesn't
support pattern synonyms.

2 years agousers_guide: Fix Docbook remnant
Ben Gamari [Thu, 8 Sep 2016 15:30:40 +0000 (11:30 -0400)] 
users_guide: Fix Docbook remnant

Not sure how this snuck through but it seems to be the only one.

2 years agoRemove unused exports
Simon Peyton Jones [Sun, 4 Sep 2016 20:58:20 +0000 (21:58 +0100)] 
Remove unused exports

2 years agoBe less picky about reporing inaccessible code
Simon Peyton Jones [Fri, 9 Sep 2016 16:42:42 +0000 (17:42 +0100)] 
Be less picky about reporing inaccessible code

Triggered by the discussion on Trac #12466, this patch
makes GHC less aggressive about reporting an error when
there are insoluble Givens.

Being so agressive was making some libraries fail to
compile, and is arguably wrong in at least some cases.
See the discussion on the ticket.

Several test now pass when they failed before; see
the files-modified list for this patch.

2 years agoAdd hs_try_putmvar()
Simon Marlow [Tue, 30 Aug 2016 19:55:10 +0000 (20:55 +0100)] 
Add hs_try_putmvar()

Summary:
This is a fast, non-blocking, asynchronous, interface to tryPutMVar that
can be called from C/C++.

It's useful for callback-based C/C++ APIs: the idea is that the callback
invokes hs_try_putmvar(), and the Haskell code waits for the callback to
run by blocking in takeMVar.

The callback doesn't block - this is often a requirement of
callback-based APIs.  The callback wakes up the Haskell thread with
minimal overhead and no unnecessary context-switches.

There are a couple of benchmarks in
testsuite/tests/concurrent/should_run.  Some example results comparing
hs_try_putmvar() with using a standard foreign export:

    ./hs_try_putmvar003 1 64 16 100 +RTS -s -N4     0.49s
    ./hs_try_putmvar003 2 64 16 100 +RTS -s -N4     2.30s

hs_try_putmvar() is 4x faster for this workload (see the source for
hs_try_putmvar003.hs for details of the workload).

An alternative solution is to use the IO Manager for this.  We've tried
it, but there are problems with that approach:
* Need to create a new file descriptor for each callback
* The IO Manger thread(s) become a bottleneck
* More potential for things to go wrong, e.g. throwing an exception in
  an IO Manager callback kills the IO Manager thread.

Test Plan: validate; new unit tests

Reviewers: niteria, erikd, ezyang, bgamari, austin, hvr

Subscribers: thomie

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

2 years agoFix TH ppr output for list comprehensions with only one Stmt
Ryan Scott [Sun, 11 Sep 2016 15:10:36 +0000 (11:10 -0400)] 
Fix TH ppr output for list comprehensions with only one Stmt

A folow-up to D2521 (which addressed #12583), where the `Outputable` `ppr`
output was tweaked to display a list comprehension with only one `Stmt` as
`[Foo]` instead of `[Foo|]` (which isn't valid Haskell). I forgot to update
the corresponding code in `Language.Haskell.TH.Ppr` which pretty-prints
`CompE`, however, so this commit takes care of that.

2 years agoFix derived Ix instances for one-constructor GADTs
Ryan Scott [Sun, 11 Sep 2016 14:27:36 +0000 (10:27 -0400)] 
Fix derived Ix instances for one-constructor GADTs

Summary:
Standalone-derived `Ix` instances would panic on GADTs with exactly
one constructor, since the list of fields was being passed to a function that
uses `foldl1` in order to generate an implementation for `inRange`. This adds a
simple check that makes `inRange` be `True` whenever a product type has no
fields.

Fixes #12583.

Test Plan: make test TEST=12583

Reviewers: simonpj, austin, bgamari

Reviewed By: bgamari

Subscribers: thomie

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

GHC Trac Issues: #12583

2 years agoRemove uses of mkMatchGroupName
Matthew Pickering [Sun, 11 Sep 2016 13:35:27 +0000 (14:35 +0100)] 
Remove uses of mkMatchGroupName

You can now just use `mkMatchGroup`.

2 years agoGeneralise type of mkMatchGroup to unify with mkMatchGroupName
Matthew Pickering [Sun, 11 Sep 2016 13:33:14 +0000 (14:33 +0100)] 
Generalise type of mkMatchGroup to unify with mkMatchGroupName

2 years agoAdd platform warning to Foreign.C.Types
Tamar Christina [Sat, 10 Sep 2016 12:04:19 +0000 (13:04 +0100)] 
Add platform warning to Foreign.C.Types

Summary:
The generated documentation for thhe Foreign.C.Types
module is generated based on the platform which ran Haddock.

This is generating incorrect types for e.g. Windows.

Add a disclaimer to the top of the page to ask people to
keep this in mind.

Test Plan: make documentation and inspect Haddock

Reviewers: erikd, austin, hvr, bgamari

Reviewed By: erikd

Subscribers: RyanGlScott, #ghc_windows_task_force, thomie

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

GHC Trac Issues: #12586

2 years agoRemove -flocal-ghci-history from default flags
Ryan Scott [Sat, 10 Sep 2016 17:51:23 +0000 (13:51 -0400)] 
Remove -flocal-ghci-history from default flags

Summary:
D2461 seemed to (inadvertently, I think) add the
`-flocal-ghci-history` flag to the list of `defaultFlags` that are enabled
automatically. As a result, every invocation of `ghci` caused a local GHCi
history to be saved to the current directory, which probably shouldn't be the
default.

Test Plan:
Run `ghci`, observe the lack of a `.ghci_history` file in your
working directory

Reviewers: austin, thomie, bgamari

Reviewed By: bgamari

Subscribers: ak3n

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

GHC Trac Issues: #9089

2 years agoMake start address of `osReserveHeapMemory` tunable via command line -xb
Francesco Mazzoli [Fri, 9 Sep 2016 17:15:49 +0000 (18:15 +0100)] 
Make start address of `osReserveHeapMemory` tunable via command line -xb

Summary:
We stumbled upon a case where an external library (OpenCL) does not work
if a specific address (0x200000000) is taken.

It so happens that `osReserveHeapMemory` starts trying to mmap at 0x200000000:

```
        void *hint = (void*)((W_)8 * (1 << 30) + attempt * BLOCK_SIZE);
        at = osTryReserveHeapMemory(*len, hint);
```

This makes it impossible to use Haskell programs compiled with GHC 8
with C functions that use OpenCL.

See this example ​https://github.com/chpatrick/oclwtf for a repro.

This patch allows the user to work around this kind of behavior outside
our control by letting the user override the starting address through an
RTS command line flag.

Reviewers: bgamari, Phyx, simonmar, erikd, austin

Reviewed By: Phyx, simonmar

Subscribers: rwbarton, thomie

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

2 years agoAdd hook for creating ghci external interpreter
Alan Zimmerman [Thu, 8 Sep 2016 06:59:48 +0000 (08:59 +0200)] 
Add hook for creating ghci external interpreter

Summary:
The external interpreter is launched by calling
'System.Process.createProcess' with a 'CreateProcess' parameter.

The current value for this has the 'std_in', 'std_out' and 'std_err'
fields use the default of 'Inherit', meaning that the remote interpreter
shares the stdio with the original ghc/ghci process.

This patch introduces a new hook to the DynFlags, which has an
opportunity to override the 'CreateProcess' fields, launch the process,
and retrieve the stdio handles actually used.

So if a ghci external interpreter session is launched from the GHC API
the stdio can be redirected if required, which is useful for tooling/IDE
integration.

Test Plan: ./validate

Reviewers: austin, hvr, simonmar, bgamari

Reviewed By: simonmar, bgamari

Subscribers: thomie

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

2 years agoUpdate unix submodule to latest HEAD.
Edward Z. Yang [Thu, 8 Sep 2016 19:05:30 +0000 (12:05 -0700)] 
Update unix submodule to latest HEAD.

Fixes readdir validation error (fixes #12572).

Signed-off-by: Edward Z. Yang <ezyang@cs.stanford.edu>