[Git][ghc/ghc][wip/marge_bot_batch_merge_job] 14 commits: RTS configure: Move over mem management checks

Marge Bot (@marge-bot) gitlab at gitlab.haskell.org
Thu Oct 12 05:08:12 UTC 2023



Marge Bot pushed to branch wip/marge_bot_batch_merge_job at Glasgow Haskell Compiler / GHC


Commits:
b7df0732 by John Ericson at 2023-10-11T16:02:11-04:00
RTS configure: Move over mem management checks

These are for heap allocation, a strictly RTS concern.

All of this should boil down to `AC_DEFINE` not `AC_SUBST`, so it
belongs in the RTS configure and should be safe to move without
modification.

The RTS configure one has a new
```
AC_CHECK_SIZEOF([void *])
```
that the top-level configure version didn't have, so that
`ac_cv_sizeof_void_p` is defined. Once more code is moved over in latter
commits, that can go away.

Progress towards #17191

- - - - -
41130a65 by John Ericson at 2023-10-11T16:02:11-04:00
RTS configure: Move over `__thread` check

This used by (@bgamari thinks) the `GCThread` abstraction in the RTS.

All of this should boil down to `AC_DEFINE` not `AC_SUBST`, so it
belongs in the RTS configure and should be safe to move without
modification.

Progress towards #17191

- - - - -
cc5ec2bd by John Ericson at 2023-10-11T16:02:11-04:00
RTS configure: Move over misc function checks

These are for general use in the RTS.

All of this should boil down to `AC_DEFINE` not `AC_SUBST`, so it
belongs in the RTS configure and should be safe to move without
modification.

Progress towards #17191

- - - - -
809e7c2d by John Ericson at 2023-10-11T16:02:11-04:00
RTS configure: Move over `eventfd` check

This check is for the RTS part of the event manager and has a
corresponding part in `base`.

All of this should boil down to `AC_DEFINE` not `AC_SUBST`, so it
belongs in the RTS configure and should be safe to move without
modification.

Progress towards #17191

- - - - -
58f3babf by John Ericson at 2023-10-11T16:02:48-04:00
Split `FP_CHECK_PTHREADS` and move part to RTS configure

`NEED_PTHREAD_LIB` is unused since
3609340743c1b25fdfd0e18b1670dac54c8d8623 (part of the make build
system), and so is no longer defined.

Progress towards #17191

- - - - -
e99cf237 by Moritz Angermann at 2023-10-11T16:03:24-04:00
nativeGen: section flags for .text$foo only

Commit 3ece9856d157c85511d59f9f862ab351bbd9b38b, was supposed to fix
#22834 in !9810.

It does however add "xr" indiscriminatly to .text sections
even if splitSections is disabled. This leads to the assembler saying:

ghc_1.s:7849:0: error:
     Warning: Ignoring changed section attributes for .text
     |
7849 | .section .text,"xr"
     | ^

- - - - -
f383a242 by Sylvain Henry at 2023-10-11T16:04:04-04:00
Modularity: pass TempDir instead of DynFlags (#17957)

- - - - -
65720412 by Ben Gamari at 2023-10-12T01:08:00-04:00
base: Introduce move modules into src

The only non-move changes here are whitespace changes to pass the
`whitespace` test and a few testsuite adaptations.

- - - - -
19032e6e by Moritz Angermann at 2023-10-12T01:08:01-04:00
CgUtils.fixStgRegStmt respect register width

This change ensure that the reg + offset computation is always of the
same size.  Before this we could end up with a 64bit register, and then
add a 32bit offset (on 32bit platforms).  This not only would fail type
sanity checking, but also incorrectly truncate 64bit values into 32bit
values silently on 32bit architectures.

- - - - -
233fab61 by Sylvain Henry at 2023-10-12T01:08:01-04:00
Apply 1 suggestion(s) to 1 file(s)
- - - - -
b0f33b6c by Moritz Angermann at 2023-10-12T01:08:01-04:00
Update CgUtils.hs
- - - - -
c84b806c by Moritz Angermann at 2023-10-12T01:08:01-04:00
Update CgUtils.hs
- - - - -
ce9c5115 by Moritz Angermann at 2023-10-12T01:08:01-04:00
Update CgUtils.hs
- - - - -
c5280857 by Moritz Angermann at 2023-10-12T01:08:01-04:00
[PEi386 linker] Bounds check and null-deref guard

We should resonably be able to expect that we won't exceed the number of
sections if we assume to be dealing with legal object files. We can however
not guarantee that we get some negative values, and while we try to
special case most, we should exclude negative indexing into the sections
array.

We also need to ensure that we do not try to derefences targetSection,
if it is NULL, due to the switch statement.

- - - - -


30 changed files:

- compiler/GHC/CmmToAsm/Ppr.hs
- compiler/GHC/StgToCmm/CgUtils.hs
- compiler/GHC/SysTools/Process.hs
- compiler/GHC/SysTools/Tasks.hs
- configure.ac
- libraries/base/base.cabal
- libraries/base/Control/Applicative.hs → libraries/base/src/Control/Applicative.hs
- libraries/base/Control/Arrow.hs → libraries/base/src/Control/Arrow.hs
- libraries/base/Control/Category.hs → libraries/base/src/Control/Category.hs
- libraries/base/Control/Concurrent.hs → libraries/base/src/Control/Concurrent.hs
- libraries/base/Control/Concurrent.hs-boot → libraries/base/src/Control/Concurrent.hs-boot
- libraries/base/Control/Concurrent/Chan.hs → libraries/base/src/Control/Concurrent/Chan.hs
- libraries/base/Control/Concurrent/MVar.hs → libraries/base/src/Control/Concurrent/MVar.hs
- libraries/base/Control/Concurrent/QSem.hs → libraries/base/src/Control/Concurrent/QSem.hs
- libraries/base/Control/Concurrent/QSemN.hs → libraries/base/src/Control/Concurrent/QSemN.hs
- libraries/base/Control/Exception.hs → libraries/base/src/Control/Exception.hs
- libraries/base/Control/Exception/Base.hs → libraries/base/src/Control/Exception/Base.hs
- libraries/base/Control/Monad.hs → libraries/base/src/Control/Monad.hs
- libraries/base/Control/Monad/Fail.hs → libraries/base/src/Control/Monad/Fail.hs
- libraries/base/Control/Monad/Fix.hs → libraries/base/src/Control/Monad/Fix.hs
- libraries/base/Control/Monad/IO/Class.hs → libraries/base/src/Control/Monad/IO/Class.hs
- libraries/base/Control/Monad/Instances.hs → libraries/base/src/Control/Monad/Instances.hs
- libraries/base/Control/Monad/ST.hs → libraries/base/src/Control/Monad/ST.hs
- libraries/base/Control/Monad/ST/Imp.hs → libraries/base/src/Control/Monad/ST/Imp.hs
- libraries/base/Control/Monad/ST/Lazy.hs → libraries/base/src/Control/Monad/ST/Lazy.hs
- libraries/base/Control/Monad/ST/Lazy/Imp.hs → libraries/base/src/Control/Monad/ST/Lazy/Imp.hs
- libraries/base/Control/Monad/ST/Lazy/Safe.hs → libraries/base/src/Control/Monad/ST/Lazy/Safe.hs
- libraries/base/Control/Monad/ST/Lazy/Unsafe.hs → libraries/base/src/Control/Monad/ST/Lazy/Unsafe.hs
- libraries/base/Control/Monad/ST/Safe.hs → libraries/base/src/Control/Monad/ST/Safe.hs
- libraries/base/Control/Monad/ST/Strict.hs → libraries/base/src/Control/Monad/ST/Strict.hs


The diff was not included because it is too large.


View it on GitLab: https://gitlab.haskell.org/ghc/ghc/-/compare/bc87484224c08772bc78a0ba774c95a9665f1c3f...c5280857da2c4a8a7a9d8056569648ce7ae60bb1

-- 
View it on GitLab: https://gitlab.haskell.org/ghc/ghc/-/compare/bc87484224c08772bc78a0ba774c95a9665f1c3f...c5280857da2c4a8a7a9d8056569648ce7ae60bb1
You're receiving this email because of your account on gitlab.haskell.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.haskell.org/pipermail/ghc-commits/attachments/20231012/067d7192/attachment.html>


More information about the ghc-commits mailing list