Zubin authored 2 years ago and Matthew Pickering committed 2 years ago

(cherry picked from commit c9967d13)

Otherwise, when installing from a bindist the C flag isn't passed to the
C compiler.

This completes the fix for #22429

(cherry picked from commit 4724e8d1)

This mirrors some existing logic for the bootstrap_target which
influences how TargetPlatform is set.

As described on #21970 not storing this led to `LlvmTarget` being set incorrectly
and hence the wrong `--target` flag being passed to the C compiler.

Towards #21970

(cherry picked from commit 64286132)

This reverts commit 7c6de18d.

Centos7 using a very old version of the toolchain (autotools-2.69) where
the behaviour of these macros has not yet changed. I am reverting this
without haste as it is blocking the 9.6 branch.

Fixes #22704

(cherry picked from commit b2a2db04)

Ben Gamari authored 2 years ago and Marge Bot committed 2 years ago

As noted in #22566, this macro is deprecated as of autoconf-2.70
`AC_PROG_CC` now sets `ac_cv_prog_cc_c99` itself.

Closes #22566.

Cheng Shao authored 2 years ago and Marge Bot committed 2 years ago

This patch fixes the bindist autoconf logic to properly set
CrossCompiling=YES when it's a cross GHC bindist.

Ben Gamari authored 2 years ago and Marge Bot committed 2 years ago

This has not worked for many, many years and relied on the now-removed
`make`-based build system.

Ultimately we want to drop mk/config.mk so here I extract the bits
needed by the Hadrian bindist installation logic into a Hadrian-specific
file. While doing this I fixed binary distribution installation, #21901.

Ben Gamari authored 2 years ago and Marge Bot committed 2 years ago

Previously we would use plain `cp` to install various parts of the
binary distribution. However, `cp`'s behavior w.r.t. file attributes is
quite unclear; for this reason it is much better to rather use
`install`.

Fixes #21965.

Ben Gamari authored 2 years ago and Marge Bot committed 2 years ago

* Rename `docs` to `doc`
* Place pdf documentation in `doc/` instead of `doc/pdfs/`

Fixes #21164.

Matthew Pickering authored 2 years ago and Marge Bot committed 2 years ago

The motiviation for this flag is to be able to produce documentation
which is suitable for uploading for hackage, ie, the cross-package links
work correctly.

There are basically three values you want to set this to:

* off - default, base_url = ../%pkg% which works for local browsing
* on - no argument , base_url = https:://hackage.haskell.org/package/%pkg%/docs - for hackage docs upload
* on - argument, for example, base_url = http://localhost:8080/package/%pkg%/docs for testing the documentation.

The `%pkg%` string is a template variable which is replaced with the
package identifier for the relevant package.

This is one step towards fixing #21749

Ben Gamari authored 2 years ago and Marge Bot committed 2 years ago

We previously only checked the stage 1/2 compiler
for --target support. We got away with this for quite a while but it
eventually caught up with us in #21579, where `bytestring`'s new NEON
implementation was unbuildable on Darwin due to Rosetta's seemingly
random logic for determining which executable image to execute. This
lead to a confusing failure to build `bytestring`'s cbits, when `clang`
tried to compile NEON builtins while targetting x86-64.

Fix this by checking CC_STAGE0 for --target support.

Fixes #21579.

Ben Gamari authored 2 years ago and Marge Bot committed 2 years ago

Here we introduce a new "virtual" package into the initial package
database, `system-cxx-std-lib`. This gives users a convenient, platform
agnostic way to link against C++ libraries, addressing #20010.

Fixes #20010.

Ben Gamari authored 2 years ago and Marge Bot committed 2 years ago

Here we introduce proper support for compilation of C++ objects. This
includes:

 * logic in `configure` to detect the C++ toolchain and propagating this
   information into the `settings` file
 * logic in the driver to use the C++ toolchain when compiling C++
   sources

Ben Gamari authored 2 years ago and Marge Bot committed 2 years ago

Since there may be .o files which are in fact archives.

Get rid of `USE_INPLACE_MINGW_TOOLCHAIN` and use a settings file entry
instead.

The CPP setting was originally introduced in f065b6b0.

John Ericson authored 3 years ago and Marge Bot committed 3 years ago

This will allow better reuse of it, such as in the upcoming RTS
configure script.

Progress towards #17191

Ben Gamari authored 3 years ago and Marge Bot committed 3 years ago

Previously the logic which called ghc-pkg failed to account for the fact
that the executable name may be prefixed with a triple. Moreover, the
call must occur before we delete the settings file as ghc-pkg needs the
latter.

Fixes #20267.

Ben Gamari authored 3 years ago and Marge Bot committed 3 years ago

Sadly, autoconf cannot warn when it encounters an undefined macro and
therefore this bug went unnoticed for altogether far too long.

Sylvain Henry authored 3 years ago and Marge Bot committed 3 years ago

The compiler should be independent of the target.

Ben Gamari authored 3 years ago and Marge Bot committed 3 years ago

None of the configure options defined by `FP_GMP` are applicable to
binary distributions.

Ben Gamari authored 4 years ago and Marge Bot committed 3 years ago

Previously we would support only one LLVM major version. Here we
generalize this to accept a range, taking this range to be LLVM 10 to 11,
as 11 is necessary for Apple M1 support. We also accept 12, as that is
what apple ships with BigSur on the M1.

Ben Gamari authored 4 years ago and Marge Bot committed 4 years ago

Previously to merge a set of object files we would invoke the linker as
usual, adding -r to the command-line. However, this can result in
non-sensical command-lines which causes lld to balk (#17962).

To avoid this we introduce a new tool setting into GHC, -pgmlm, which is
the linker which we use to merge object files.

Sylvain Henry authored 4 years ago and Marge Bot committed 4 years ago

* Represent backends with a `Backend` datatype in GHC.Driver.Backend

* Don't detect the default backend to use for the target platform at
  compile time in Hadrian/make but at runtime. It makes "Settings"
  simpler and it is a step toward making GHC multi-target.

* The latter change also fixes hadrian which has not been updated to
  take into account that the NCG now supports AIX and PPC64 (cf
  df26b955 and
  d3c1dda6)

* Also we don't treat iOS specifically anymore (cf
  cb4878ff)

Sylvain Henry authored 5 years ago and Marge Bot committed 4 years ago

Metric Decrease:
    T13035
    T1969

Ben Gamari authored 5 years ago and Marge Bot committed 5 years ago

In addition, we prefer the Mingw64 Python distribution on Windows due
to #17483.

Ben Gamari authored 5 years ago and Marge Bot committed 5 years ago

Fixing #17255.

Ben Gamari authored 5 years ago and Marge Bot committed 5 years ago

This had silently regressed due to 81860281 and the variable renaming performed
in b55ee979, as noted in #17374.

John Ericson authored 5 years ago and Marge Bot committed 5 years ago

Some where using `True` / `False`, a legacy of when they were in
`Config.hs`. See #16914 / d238d306 for a similar problem.

Also clean up the configure variables names for consistency and clarity
while we're at it. "Target" makes clear we are talking about outputted
code, not where GHC itself runs.