[Git][ghc/ghc][wip/marge_bot_batch_merge_job] 6 commits: Add -Wnoncanonical-{monad,monoid}-instances to standardWarnings

Marge Bot gitlab at gitlab.haskell.org
Thu Oct 15 01:40:20 UTC 2020



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


Commits:
e60ae8a3 by Fumiaki Kinoshita at 2020-10-14T18:06:12-04:00
Add -Wnoncanonical-{monad,monoid}-instances to standardWarnings

-------------------------
Metric Decrease:
   T12425
Metric Increase:
   T17516
-------------------------

- - - - -
15d2340c by Simon Peyton Jones at 2020-10-14T18:06:48-04:00
Fix some missed opportunities for preInlineUnconditionally

There are two signficant changes here:

* Ticket #18815 showed that we were missing some opportunities for
  preInlineUnconditionally.  The one-line fix is in the code for
  GHC.Core.Opt.Simplify.Utils.preInlineUnconditionally, which now
  switches off only for INLINE pragmas.  I expanded
  Note [Stable unfoldings and preInlineUnconditionally] to explain.

* When doing this I discovered a way in which preInlineUnconditionally
  was occasionally /too/ eager.  It's all explained in
  Note [Occurrences in stable unfoldings] in GHC.Core.Opt.OccurAnal,
  and the one-line change adding markAllMany to occAnalUnfolding.

I also got confused about what NoUserInline meant, so I've renamed
it to NoUserInlinePrag, and changed its pretty-printing slightly.
That led to soem error messate wibbling, and touches quite a few
files, but there is no change in functionality.

I did a nofib run.  As expected, no significant changes.

        Program           Size    Allocs
----------------------------------------
         sphere          -0.0%     -0.4%
----------------------------------------
            Min          -0.0%     -0.4%
            Max          -0.0%     +0.0%
 Geometric Mean          -0.0%     -0.0%

I'm allowing a max-residency increase for T10370, which seems
very irreproducible. (See comments on !4241.)  There is always
sampling error for max-residency measurements; and in any case
the change shows up on some platforms but not others.

Metric Increase:
    T10370

- - - - -
0c4bfed8 by Ben Gamari at 2020-10-14T18:07:25-04:00
users-guide: Add missing :ghc-flag: directive
- - - - -
d029503c by Krzysztof Gogolewski at 2020-10-14T21:40:11-04:00
Remove Proxy# argument in Data.Typeable.Internal

No longer neccessary - TypeRep is now indexed, there is no ambiguity.

Also fix a comment in Evidence.hs, IsLabel no longer takes a Proxy#.

- - - - -
62192308 by Sylvain Henry at 2020-10-14T21:40:15-04:00
Fix parsing of PIE flags

-fPIE and -fno-PIE flags were (un)setting Opt_PIC instead of Opt_PIE.

Original commit: 3625728a0e3a9b56c2b85ae7ea8bcabdd83ece6a

- - - - -
fa362f10 by Ben Gamari at 2020-10-14T21:40:16-04:00
testsuite: Add missing #include on <stdlib.h>

This otherwise fails on newer Clangs, which warn
more aggressively on undeclared symbols.
- - - - -


30 changed files:

- compiler/GHC/Core/Opt/OccurAnal.hs
- compiler/GHC/Core/Opt/Simplify.hs
- compiler/GHC/Core/Opt/Simplify/Utils.hs
- compiler/GHC/Core/Opt/Specialise.hs
- compiler/GHC/Core/Opt/WorkWrap.hs
- compiler/GHC/Driver/Session.hs
- compiler/GHC/Hs/Binds.hs
- compiler/GHC/HsToCore/Binds.hs
- compiler/GHC/HsToCore/Quote.hs
- compiler/GHC/Parser.y
- compiler/GHC/Rename/Module.hs
- compiler/GHC/Tc/Instance/Class.hs
- compiler/GHC/Tc/Types/Evidence.hs
- compiler/GHC/ThToHs.hs
- compiler/GHC/Types/Basic.hs
- compiler/GHC/Utils/Binary.hs
- docs/users_guide/9.0.1-notes.rst
- docs/users_guide/phases.rst
- libraries/base/Data/Typeable/Internal.hs
- libraries/base/GHC/Exception.hs-boot
- testsuite/tests/codeGen/should_compile/jmp_tbl.hs
- testsuite/tests/concurrent/should_run/conc059_c.c
- testsuite/tests/cpranal/should_compile/Cpr001_imp.hs
- testsuite/tests/deriving/should_compile/drv020.hs
- testsuite/tests/determinism/determ019/A.hs
- testsuite/tests/gadt/gadt16.hs
- testsuite/tests/gadt/nbe.hs
- testsuite/tests/ghci.debugger/HappyTest.hs
- testsuite/tests/ghci/scripts/T4127.script
- testsuite/tests/ghci/scripts/T4127.stdout


The diff was not included because it is too large.


View it on GitLab: https://gitlab.haskell.org/ghc/ghc/-/compare/9a5fa43b90508d11cc8a4f87f91ec6b95da685a8...fa362f102215f335b98b23a45772c33363122d1a

-- 
View it on GitLab: https://gitlab.haskell.org/ghc/ghc/-/compare/9a5fa43b90508d11cc8a4f87f91ec6b95da685a8...fa362f102215f335b98b23a45772c33363122d1a
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/20201014/19d6b92a/attachment-0001.html>


More information about the ghc-commits mailing list