- 25 Jul, 2014 4 commits
-
-
Joachim Breitner authored
after changes in 92587bfe. This problem was noticed on ghcspeed (although only by accident, unfortunately, as a change from 0 to 1 is not reported in the summary).
-
Simon Peyton Jones authored
This patch was provoked by Trac #5610, which I finally got a moment to look at. In the end I added a new data type ErrUtils.Validity, data Validity = IsValid -- Everything is fine | NotValid MsgDoc -- A problem, and some indication of why with some suitable combinators, and used it where appropriate (which touches quite a few modules). The main payoff is that error messages improve for FFI type validation.
-
Edward Z. Yang authored
Signed-off-by:Edward Z. Yang <ezyang@cs.stanford.edu>
-
Simon Peyton Jones authored
Fixes Trac #9357
-
- 24 Jul, 2014 2 commits
-
-
Simon Peyton Jones authored
This patch fixes Trac #9359
-
Duplicate record fields would not be detected when given a type with multiple data constructors, and the first data constructor had a record field r1 and any consecutive data constructors had multiple fields named r1. This fixes #9156 and was reviewed in https://phabricator.haskell.org/D87
-
- 23 Jul, 2014 7 commits
-
-
tibbe authored
-
tibbe authored
The test was incorrectly testing that NAND is associative, which it isn't.
-
tibbe authored
Before LOCK was a separate instruction and this led to the register allocator separating it from the instruction it was supposed to be a prefix of, leading to illegal assembly such as lock mov Fix contributed by PÁLI Gábor János. -
Gabor Greif authored
-
Edward Z. Yang authored
Signed-off-by: -
Edward Z. Yang authored
Signed-off-by: -
Edward Z. Yang authored
Signed-off-by:
-
- 22 Jul, 2014 3 commits
-
-
Edward Z. Yang authored
Signed-off-by: -
Edward Z. Yang authored
Signed-off-by: -
Joachim Breitner authored
Summary: Make last a good consumer simply by implementing it as foldl. This fixes Trac: #9339. Thanks to David Feuer for bringing it up. Test Plan: perf/should_run/T9339 + general validation Reviewers: austin Reviewed By: austin Subscribers: phaskell, simonmar, relrod, carter Differential Revision: https://phabricator.haskell.org/D86 Trac Issues: #9339
-
- 21 Jul, 2014 13 commits
-
-
pali.gabor@gmail.com authored
The recent version of ghc-prim assumes a more modern processor as it exploits built-in atomic operations, and some of them are not yet present on i486. Hence the -march flag is explicitly set to i686 for the C compiler -- just to be in sync with the default of other distributions.
-
Edward Z. Yang authored
Signed-off-by: -
Edward Z. Yang authored
This reverts commit f23b2129.
-
Edward Z. Yang authored
Signed-off-by: -
Edward Z. Yang authored
Summary: Previously, both Cabal and GHC defined the type PackageId, and we expected them to be roughly equivalent (but represented differently). This refactoring separates these two notions. A package ID is a user-visible identifier; it's the thing you write in a Cabal file, e.g. containers-0.9. The components of this ID are semantically meaningful, and decompose into a package name and a package vrsion. A package key is an opaque identifier used by GHC to generate linking symbols. Presently, it just consists of a package name and a package version, but pursuant to #9265 we are planning to extend it to record other information. Within a single executable, it uniquely identifies a package. It is *not* an InstalledPackageId, as the choice of a package key affects the ABI of a package (whereas an InstalledPackageId is computed after compilation.) Cabal computes a package key for the package and passes it to GHC using -package-name (now *extremely* misnamed). As an added bonus, we don't have to worry about shadowing anymore. As a follow on, we should introduce -current-package-key having the same role as -package-name, and deprecate the old flag. This commit is just renaming. The haddock submodule needed to be updated. Signed-off-by:
-
Austin Seipp authored
Signed-off-by:Austin Seipp <austin@well-typed.com>
-
Austin Seipp authored
Signed-off-by: -
Austin Seipp authored
Signed-off-by: -
Herbert Valerio Riedel authored
The `time` package has recently moved its primary upstream location to https://github.com/haskell/time. This leaves `transformers` the only package used by GHC still requiring darcs2git mirroring machinery.
-
Edward Z. Yang authored
Summary: Previously, the full database stack was used for ghc-pkg to modify packages, which meant that commands like 'ghc-pkg unregister --user' worked the same as 'ghc-pkg unregister'. Since package modification is a "read and write" operation, we should use the flag db stack (which is currently used for reads) to determine which database to update. There is also a new flag --user-package-db, which lets you explicitly set the user database (as seen by --user). This was mostly added to aid in testing, but could be useful for end users as well. Signed-off-by:
-
Edward Z. Yang authored
Summary: Previously, the GHC API was "semi" wired-in: it was installed with a version number, but that version number was hard-coded into the compiler and it wasn't really possible to install other copies of the GHC API. This patch makes the GHC API more similar to existing wired-in packages such as ghc-prim, and will be helpful when we start extending the amount of information passed to -package-name. Signed-off-by:
-
Gabor Greif authored
-
Joachim Breitner authored
This reverts commit c41b716d. With the Cabal bump reverted, this should build again, and we aim to keep it buildable like this.
-
- 20 Jul, 2014 11 commits
-
-
Austin Seipp authored
Signed-off-by: -
Austin Seipp authored
Signed-off-by: -
Austin Seipp authored
Signed-off-by: -
Austin Seipp authored
Patch contributed by slowmo. Signed-off-by: -
bernalex authored
Test Plan: validate Reviewers: austin Reviewed By: austin Subscribers: phaskell, simonmar, relrod, carter Differential Revision: https://phabricator.haskell.org/D77
-
Sergei Trofimovich authored
Summary: Signed-off-by: Sergei Trofimovich <slyfox@gentoo.org> Test Plan: git grep, git log -SSpinLockCount, build test Reviewers: austin, simonmar Reviewed By: austin, simonmar Subscribers: phaskell, simonmar, relrod, carter Differential Revision: https://phabricator.haskell.org/D76
-
Mathieu Boespflug authored
Summary: As a security precaution, GHCi helpfully refuses to run a .ghci file if it is owned by another user. But if the that other user is root, then arguably GHCi should not refuse to interpret the file, because if root really was malicious, then the user would be having a bad day anyways. This means that .ghci files installed in a global location, say under /usr/local/, can now be read. Fixes #9324 Test Plan: ``` $ sudo touch .ghci $ ghci ``` Notice that the warning about the file being owned by someone else is now gone. Reviewers: austin Reviewed By: austin Subscribers: phaskell, simonmar, carter, nomeata, relrod Projects: #ghc Differential Revision: https://phabricator.haskell.org/D75
-
kgardas authored
Summary: This patch fixes inconsistency in exported functions from TcSplice.lhs and TcSplice.lhs-boot files. It looks like only GHC HEAD is sensitive to it and complains about it while bootstraping another HEAD. At least this is what happening on Solaris/AMD64 builder machine where GHC 7.9.20140620 is used as a boostrap compiler. The failure does not happen on another builders. Test Plan: validate Reviewers: austin Reviewed By: austin Subscribers: phaskell, simonmar, relrod, carter Differential Revision: https://phabricator.haskell.org/D74
-
agibiansky authored
Summary: Add a few exports to be generated by the Happy parser module. Add documentation showing how to use the Happy parser. Test Plan: Validate Reviewers: carter, austin Reviewed By: austin Subscribers: phaskell, simonmar, relrod, carter Differential Revision: https://phabricator.haskell.org/D71
-
bernalex authored
Summary: Carl Howells pointed out[0] that the `Monoid` instance for `Data.Proxy.Proxy` is only defined for types with kind *. This is a very mild change. Furthermore, Edward Kmett revealed[1] that it was supposed to be there all along -- the extension simply got lost in commit 1d1ff77a, as pointed out by Adam Vogt[2]. This used to be correct in GHC 7.6, so this commit fixes a regression. This addresses #9317. [0] <http://www.haskell.org/pipermail/libraries/2014-July/023261.html>. [1] <http://www.haskell.org/pipermail/libraries/2014-July/023267.html>. [2] <http://www.haskell.org/pipermail/libraries/2014-July/023265.html >. Signed-off-by:
Alexander Berntsen <alexander@plaimi.net> Test Plan: See [0] Reviewers: austin, hvr, ekmett Reviewed By: austin, hvr, ekmett Subscribers: phaskell, simonmar, relrod, carter Differential Revision: https://phabricator.haskell.org/D70
-
Austin Seipp authored
Signed-off-by:
-
