MERGED: Set interfacedir (using $topdir, not $httptopdir)
Mon Oct 29 10:48:25 PDT 2007  Ian Lynagh <igloo@earth.li>

Teach ghc-pkg about $httptopdir

MERGED: installPackage needs to treat $httptopdir the same as $topdir
Sun Oct 28 06:45:34 PDT 2007  Ian Lynagh <igloo@earth.li>

MERGED: Define and use $httptopdir for the haddock docs locations
Sun Oct 28 05:35:52 PDT 2007  Ian Lynagh <igloo@earth.li>

We need to copy .buildinfo files into the bindists

(>>>) now comes from GHC.Desugar

Refactor error recovery slightly

Mostly this patch is refacoring, but it also avoids post-tc zonking if
the typechecker found errors.  This is good because otherwise with
DEBUG you can get the "Inventing strangely-kinded TyCon" warning.

Avoid Haddock bug #1821

Update error message to mention -XPatternSignatures instead of -fglasgow-exts

Rejig the error messages a bit; fixes a minor bug

The type checker was only reporting the first message if an equality
failed to match.  This patch does a bit of refactoring and fixes the
bug, which was in the bogus use of eqInstMisMatch
in tcSimplify.report_no_instances.b

This is really a bug in 6.8 too, so this would be good to merge across
to the 6.8 branch.

Refactor Haddock options

This patch renames the DOC_OPTIONS pragma to OPTIONS_HADDOCK. It also
adds "-- # ..."-style Haddock option pragmas, for compatibility with
code that use them.

Another change is that both of these two pragmas behave like
OPTIONS_GHC, i.e. they are only allowed at the top of the module, they
are ignored everywhere else and they are stored in the dynflags. There is
no longer any Haddock options in HsSyn.

Please merge this to the 6.8.2 branch when 6.8.1 is out, if appropriate.

clean ghci-inplace

clean Haddock droppings

Fix warning in OSMem for darwin

FIX BUILD: a glitch in the new rules and inlining stuff

Don't re-add the worker info to a binder until completeBind. It's not
needed in its own RHS, and it may be replaced, via the substitution
following postInlineUnconditionally.

(Fixes build of the stage2 compiler which fell over when Coercion.lhs
was being compiled.)

Fix LiberateCase

Merge to STABLE please

Liberate case was being far too gung-ho about what to specialise. This
bug only showed up when a recursive function 'f' has a nested recursive
function 'g', where 'g' calls 'f' (as well as recursively calling 'g').
This exact situation happens in GHC/IO.writeLines.

This patch puts things right; see Note [When to specialise].  Result:
much less code bloat.

Improve error-message output slightly

Improve documentation of orphan instances (thanks to Adrian Hey)

Please push to stable branch

Simon

fix installation of haddock.css and friends

In a pattern binding, a type sig in the pattern cannot bind a type variable

In a pattern binding with a pattern type signature, such as

(Just (x::a)) = e

the pattern type signature cannot bind type variables.  Hence
'a' must be in scope already for the above example to be legal.

This has been the situation for some time, but Dan changed it when
adding view patterns.  This one-line change restores the old behaviour.

Substantial improvement to the interaction of RULES and inlining

(Merge to 6.8 branch after testing.)

There were a number of delicate interactions between RULEs and inlining
in GHC 6.6.  I've wanted to fix this for a long time, and some perf
problems in the 6.8 release candidate finally forced me over the edge!

The issues are documented extensively in OccurAnal, Note [Loop breaking
and RULES], and I won't duplicate them here.  (Many of the extra lines in
OccurAnal are comments!)

This patch resolves Trac bugs #1709, #1794, #1763, I believe.

Add newline in debug print

Explicit pattern match in default case of addTickLHsBind

Generalise the types of mk_FunBind, mk_easy_FunBind, mkVarBind

Fix the build with GHC < 6.4 (foldl1' didn't exist)

Fix the build with GHC < 6.4 (foldl1' didn't exist)

MERGED: We need to install-docs when making the Windows bindist

We need to set _way=* in rts/ both when making and installing bindists
This is a hack, but it means we get libHSrts*.a etc rather than just
libHSrts.a.

Fix a whole heap of speling errrs in the docs

Only build/install the man page if XSLTPROC is defined

install the Cabal docs, and make them show up in a binary distribution

cp => $(CP)

get rid of the html subdirectory under share/doc/ghc/users_guide

Make 'improvement' work properly in TcSimplify

(Please merge this, and the preceding
handful from me to the 6.8 branch.)

This patch fixes a serious problem in the type checker, whereby
TcSimplify was going into a loop because it thought improvement
had taken place, but actually the unificataion was actually deferred.

We thereby fix Trac #1781, #1783, #1795, and #1797!

In fixing this I found what a mess TcSimplify.reduceContext is!
We need to fix this.

The main idea is to replace the "improvement flag" in Avails with
a simpler and more direct test: have any of the mutable type variables
in the (zonked) 'given' or 'irred' constraints been filled in?
This test uses the new function TcMType.isFilledMetaTyVar; the test
itself is towards the end of reduceContext.

I fixed a variety of other infelicities too, and left some ToDos.

An implication constraint can abstract over EqInsts

In an AbsBinds, the 'dicts' can include EqInsts

An AbsBinds abstrats over evidence, and the evidence can be both
Dicts (class constraints, implicit parameters) and EqInsts (equality
constraints).  So we need to
  - use varType rather than idType
  - use instToVar rather than instToId
  - use zonkDictBndr rather than zonkIdBndr in zonking

It actually all worked before, but gave warnings.

More notes

Comments only

Add anyM to IOEnv

Add a note to NOTES

Make compileToCore return the module name and type environment along with bindings

  compileToCore returned just a list of CoreBind, which isn't enough,
since to do anything with the resulting Core code, you probably also
want the type declarations. I left compileToCore as it is, but added a
function compileToCoreModule that returns a complete Core module (with
module name, type environment, and bindings). I'm not sure that
returning the type environment is the best way to represent the type
declarations for the given module, but I don't want to reinvent the
External Core wheel for this.

binary-dist: Makefile-vars needs HADDOCK_DOCS=YES

fix the links in the library documentation index

default to installing runhaskell and hsc2hs again, but provide knobs to turn them off

Adding hpc documentation about sum and map, push to STABLE.

Fixing typo in runtime documentation for hpc, push to stable

Correct a comment

Fix ghc package in bindists; it wasn't adding the depenedency on readline

Fix installing the ghc package .hi files in a bindist

Build the manpage when building, not when installing

Hack to make sure we get all the RTS ways in bindists

Fix installing the documentation in the bindists

-ftype-families -> -XTypeFamilies

FIX #1791: fail with out-of-heap when allocating more than the max heap size in one go
Normally the out-of-heap check is performed post-GC, but there are
cases where we can detect earlier that we definitely have exhausted
the heap size limit.

Fix more warnings

FIX BUILD (on 32-bit platforms): hs_hpc_module() type mismatch

patch from #1782; fixes check-packages target on Solaris

add PIC relocations for x86_64, and use a simpler hack in place of x86_64_high_symbol()
This is Wolfgang Thaller's patch sent to cvs-ghc recently, with extra
commentary by me.  It turns out that this patch is not just a cleanup,
it is also necessary for GHCi to work on x86_64 with shared libraries,
because previously lookupSymbol() was creating jump-table entries for
all symbols looked up that resolved outside 2Gb, whereas Wolfgang's
version only generates jump-table entries for 32-bit symbol references
in object code that we load.

fix creation of ghc-inplace for non-std ways

remove an incorrect assertion

second attempt to fix C compiler warnings with -fhpc
The hs_hpc_module() prototype in RtsExternal.h didn't match its usage:
we were passing StgWord-sized parameters but the prototype used C
ints.  I think it accidentally worked because we only ever passed
constants that got promoted.  The constants unfortunately were
sometimes negative, which caused the C compiler to emit warnings.

I suspect PprC.pprHexVal may be wrong to emit negative constants in
the generated C, but I'm not completely sure.  Anyway, it's easy to
fix this in CgHpc, which is what I've done.

Zonk quantified tyvars with skolems

We used to zonk quantified type variables to regular TyVars.  However, this
leads to problems.  Consider this program from the regression test suite:

  eval :: Int -> String -> String -> String
  eval 0 root actual = evalRHS 0 root actual

  evalRHS :: Int -> a
  evalRHS 0 root actual = eval 0 root actual

It leads to the deferral of an equality

  (String -> String -> String) ~ a

which is propagated up to the toplevel (see TcSimplify.tcSimplifyInferCheck).
In the meantime `a' is zonked and quantified to form `evalRHS's signature.
This has the *side effect* of also zonking the `a' in the deferred equality
(which at this point is being handed around wrapped in an implication
constraint).

Finally, the equality (with the zonked `a') will be handed back to the
simplifier by TcRnDriver.tcRnSrcDecls calling TcSimplify.tcSimplifyTop.
If we zonk `a' with a regular type variable, we will have this regular type
variable now floating around in the simplifier, which in many places assumes to
only see proper TcTyVars.

We can avoid this problem by zonking with a skolem.  The skolem is rigid
(which we requirefor a quantified variable), but is still a TcTyVar that the
simplifier knows how to deal with.

Fix typo that prevented zonking of rhs of EqInsts

MERGE TO STABLE

implement #1468, :browse on its own uses the currently-loaded module

FIX #1784: EM_AMD64 and EM_X86_64 might both be defined to the same value

Tell Cabal what it's version number is while bootstrapping it
This means that it'll work with all the packages that specify a cabal-version

FIX #1450: asynchronous exceptions are now printed by +RTS -xc

fix -fbreak-on-exception for unregsterised

fix :print when !tablesNextToCode

fix breakpoints in unregisterised mode

Change some ints to unsigned ints
Fixes some gratuitous warnings when compiling via C with -fhpc

fix warnings when compiling via C

rollback "accounting wibble: we were missing an alloc_blocks .. "
I misread the code, now added a comment to explain why it isn't necessary

recordMutable: test for gen>0 before calling recordMutableCap
For some reason the C-- version of recordMutable wasn't verifying that
the object was in an old generation before attempting to add it to the
mutable list, and this broke maessen_hashtab.  This version of
recordMutable is only used in unsafeThaw#.

re-instate missing parts of "put the @N suffix on stdcall foreign calls in .cmm code"
These changes were apparently lost during "massive changes to add a
'zipper' representation of C-"

Don't barf on error message with non-tc tyvars

MERGE TO STABLE

Fix deferring on tyvars in TcUnify.subFunTys

TcUnify.subFunTys must take type families into account
* A bug reported by Andrew Appleyard revealed that subFunTys did take
  neither type families nor equalities into account.  In a fairly obscure
  case there was also a coercion ignored.

Refactoring: extract platform-specific code from sm/MBlock.c
Also common-up some duplicate bits in the platform-specific code

fix an error message (barf -> sysErrorBelch)

fix warning on Windows

Don't clean gmp when validating (speeds up validation on Windows)

document float2Int# and double2Int#

Update HsExpr.hi-boot-6 for view pattern changes

Fix #1709: do not expose the worker for a loop-breaker

The massive 'Uni' program produced a situation in which a function that
had a worker/wrapper split was chosen as a loop breaker.  If the worker
is exposed in the interface file, then an importing module may go into
an inlining loop: see comments on TidyPgm.tidyWorker.

This patch fixes the inlining bug.  The code that gives rise to this
bizarre case is still not good (it's a bunch of implication constraints
and we are choosing a bad loop breaker) but the first thing is to fix the
bug.

It's rather hard to produce a test case!

Please merge to the 6.8 branch.

Fix #1662: do not simplify constraints for vanilla pattern matches

See Note [Arrows and patterns] in TcPat.

This fixes Trac 1662.   Test is arrows/should_compile/arrowpat.hs

Please merge

Eliminate over-zealous warning in CoreToStg

Show inlined function in the header of 'Inlining done' message

Show program size in the simplifier-bailing-out message

View patterns, record wildcards, and record puns
This patch implements three new features:
* view patterns (syntax: expression -> pat in a pattern)
* working versions of record wildcards and record puns
See the manual for detailed descriptions.

Other minor observable changes:
* There is a check prohibiting local fixity declarations
  when the variable being fixed is not defined in the same let
* The warn-unused-binds option now reports warnings for do and mdo stmts

Implementation notes:

* The pattern renamer is now in its own module, RnPat, and the
implementation is now in a CPS style so that the correct context is
delivered to pattern expressions.

* These features required a fairly major upheaval to the renamer.
Whereas the old version used to collect up all the bindings from a let
(or top-level, or recursive do statement, ...) and put them into scope
before renaming anything, the new version does the collection as it
renames.  This allows us to do the right thing with record wildcard
patterns (which need to be expanded to see what names should be
collected), and it allows us to implement the desired semantics for view
patterns in lets.  This change had a bunch of domino effects brought on
by fiddling with the top-level renaming.

* Prior to this patch, there was a tricky bug in mkRecordSelId in HEAD,
which did not maintain the invariant necessary for loadDecl.  See note
[Tricky iface loop] for details.

FIX profiling after my storage manager changes

More docu for skolemOccurs

Slightly improved comments in TcTyClsDecls

TcTyFuns: remove some duplicate code

TcTyFuns.eqInstToRewrite

Add allocateInGen() for allocating in a specific generation, and cleanups
Now allocate() is a synonym for allocateInGen().

I also made various cleanups: there is now less special-case code for
supporting -G1 (two-space collection), and -G1 now works with
-threaded.

optimise isAlive()

accounting wibble: we were missing an alloc_blocks++ in allocateLocal()

threadStackOverflow should be using allocateLocal

FIX #1759 while respecting the ticks