[Git][ghc/ghc][wip/andreask/ci_validate] 2 commits: rts: EventLog.c: Properly cast (potential) 32bit pointers to uint64_t

[Andreas Klebinger]

Andreas Klebinger pushed to branch wip/andreask/ci_validate at Glasgow Haskell Compiler / GHC


Commits:
73514d32 by Andreas Klebinger at 2020-11-25T16:36:59+01:00
rts: EventLog.c: Properly cast (potential) 32bit pointers to uint64_t

- - - - -
1e9f0d5a by Andreas Klebinger at 2020-11-25T16:36:59+01:00
Use validate flavour for all CI builds.

This also means we compile GHC with -O1 instead of -O2 for some
platforms for CI. As a result a lot of test metrics
got worse which we now have to accept.

-------------------------
Metric Increase:
    ManyAlternatives
    ManyConstructors
    MultiLayerModules
    Naperian
    T10421
    T12150
    T12227
    T12234
    T12425
    T12545
    T12707
    T13035
    T13253
    T13253-spj
    T13379
    T13719
    T14697
    T18282
    T18698a
    T18698b
    T1969
    T3064
    T3294
    T4801
    T5321FD
    T5321Fun
    T5631
    T6048
    T783
    T9020
    T9203
    T9233
    T9872a
    T9872b
    T9872c
    T9872d
    T9961
    haddock.Cabal
    haddock.base
    parsing001
-------------------------

- - - - -


2 changed files:

- .gitlab-ci.yml
- rts/eventlog/EventLog.c


Changes:

=====================================
.gitlab-ci.yml
=====================================
@@ -442,7 +442,7 @@ validate-x86_64-darwin:
     LANG: "en_US.UTF-8"
     CONFIGURE_ARGS: "--with-intree-gmp"
     TEST_ENV: "x86_64-darwin"
-    BUILD_FLAVOUR: "perf"
+    BUILD_FLAVOUR: "validate"
   after_script:
     - cp -Rf $HOME/.cabal cabal-cache
     - .gitlab/ci.sh clean
@@ -493,7 +493,7 @@ validate-x86_64-darwin:
   tags:
     - x86_64-linux
   variables:
-    BUILD_FLAVOUR: "perf"
+    BUILD_FLAVOUR: "validate"
   before_script:
       # Build hyperlinked sources for documentation when building releases
     - |


=====================================
rts/eventlog/EventLog.c
=====================================
@@ -1489,7 +1489,8 @@ static void postTickyCounterDef(EventsBuf *eb, StgEntCounter *p)
     ensureRoomForVariableEvent(eb, len);
     postEventHeader(eb, EVENT_TICKY_COUNTER_DEF);
     postPayloadSize(eb, len);
-    postWord64(eb, (uint64_t) p);
+
+    postWord64(eb, (uint64_t)((uintptr_t) p));
     postWord16(eb, (uint16_t) p->arity);
     postString(eb, p->arg_kinds);
     postString(eb, p->str);
@@ -1513,7 +1514,7 @@ static void postTickyCounterSample(EventsBuf *eb, StgEntCounter *p)
 
     ensureRoomForEvent(eb, EVENT_TICKY_COUNTER_SAMPLE);
     postEventHeader(eb, EVENT_TICKY_COUNTER_SAMPLE);
-    postWord64(eb, (uint64_t) p);
+    postWord64(eb, (uint64_t)((uintptr_t) p));
     postWord64(eb, p->entry_count);
     postWord64(eb, p->allocs);
     postWord64(eb, p->allocd);



View it on GitLab: https://gitlab.haskell.org/ghc/ghc/-/compare/cb8f1cb04177174e0338f7ad59392625488a6ef2...1e9f0d5aa9caf47c18aa4b3abd0d610959ab4d07

-- 
View it on GitLab: https://gitlab.haskell.org/ghc/ghc/-/compare/cb8f1cb04177174e0338f7ad59392625488a6ef2...1e9f0d5aa9caf47c18aa4b3abd0d610959ab4d07
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/20201125/b844d8ec/attachment-0001.html>