trommler (Peter Trommler)
User

Projects

User does not belong to any projects.

User Details

User Since
Jun 6 2014, 3:56 PM (235 w, 2 d)

Recent Activity

Yesterday

trommler updated the summary of D5323: PPC NCG: Generate MO_?_QuotRem for subword sizes.
Sun, Dec 9, 4:52 AM
trommler added a comment to D5323: PPC NCG: Generate MO_?_QuotRem for subword sizes.

Inline comments done.

Sun, Dec 9, 3:11 AM
trommler updated the diff for D5323: PPC NCG: Generate MO_?_QuotRem for subword sizes.
  • Refactor PPC remainder code
  • Rebase
Sun, Dec 9, 3:09 AM

Sat, Dec 8

trommler updated the diff for D5300: PPC NCG: Implement MachOps for smaller sizes.

Rebase

Sat, Dec 8, 8:39 AM
trommler planned changes to D5323: PPC NCG: Generate MO_?_QuotRem for subword sizes.

I plan the following

Sat, Dec 8, 3:31 AM

Wed, Dec 5

trommler added a comment to D5297: Mark broken tests on powerpc64le and skip ghci-ext.

What can I do to move this forward?

Wed, Dec 5, 2:47 PM

Tue, Nov 27

trommler added a comment to D5297: Mark broken tests on powerpc64le and skip ghci-ext.

CI keeps failing, now on Linux and OS X. I validated locally on powerpc64le and x86_64.

Actually, the former was done as part of D5300.

Tue, Nov 27, 12:25 AM
trommler added a comment to D5297: Mark broken tests on powerpc64le and skip ghci-ext.

CI keeps failing, now on Linux and OS X. I validated locally on powerpc64le and x86_64.

Tue, Nov 27, 12:24 AM

Sun, Nov 25

trommler updated the diff for D5323: PPC NCG: Generate MO_?_QuotRem for subword sizes.
  • Break up long line
  • Rebase
Sun, Nov 25, 3:33 AM
trommler updated the diff for D5300: PPC NCG: Implement MachOps for smaller sizes.

Rebase

Sun, Nov 25, 1:37 AM

Sat, Nov 24

trommler added a comment to D5297: Mark broken tests on powerpc64le and skip ghci-ext.

This validates now on x86_64, powerpc64le needs D5300.

Sat, Nov 24, 11:22 AM
trommler updated the diff for D5297: Mark broken tests on powerpc64le and skip ghci-ext.
  • rebase
Sat, Nov 24, 11:20 AM

Fri, Nov 23

trommler planned changes to D5297: Mark broken tests on powerpc64le and skip ghci-ext.

I could not validate this because HEAD does not build for me (neither x86_64 nor powerpc64le). Perhaps I need to wait for things to be fixed in HEAD and rebase again.

Fri, Nov 23, 10:31 AM
trommler updated the diff for D5297: Mark broken tests on powerpc64le and skip ghci-ext.

Rebase

Fri, Nov 23, 1:28 AM

Sun, Nov 18

trommler updated the diff for D5300: PPC NCG: Implement MachOps for smaller sizes.

Rebase

Sun, Nov 18, 7:55 AM

Sun, Nov 11

trommler added a parent revision for D5323: PPC NCG: Generate MO_?_QuotRem for subword sizes: D5300: PPC NCG: Implement MachOps for smaller sizes.
Sun, Nov 11, 4:36 AM
trommler added a child revision for D5300: PPC NCG: Implement MachOps for smaller sizes: D5323: PPC NCG: Generate MO_?_QuotRem for subword sizes.
Sun, Nov 11, 4:36 AM
trommler created D5323: PPC NCG: Generate MO_?_QuotRem for subword sizes.
Sun, Nov 11, 4:28 AM

Nov 5 2018

trommler created D5300: PPC NCG: Implement MachOps for smaller sizes.
Nov 5 2018, 5:25 AM

Nov 3 2018

trommler created D5297: Mark broken tests on powerpc64le and skip ghci-ext.
Nov 3 2018, 11:42 AM

Oct 31 2018

trommler abandoned D5273: Fix unused label on non x86_64 platforms.

I guess I can abandon this. Thanks @alpmestan for your review and discussion of the issue!

Oct 31 2018, 1:37 PM
trommler added a comment to D5273: Fix unused label on non x86_64 platforms.

I think this is fixed in HEAD now with commit 82a7164

Oct 31 2018, 1:35 PM

Oct 29 2018

trommler added a comment to D5273: Fix unused label on non x86_64 platforms.

A whole lot of test failures, see https://circleci.com/gh/ghc/ghc-diffs/484;

Oct 29 2018, 6:25 AM

Oct 28 2018

trommler added a comment to D5230: rts/Linker.c: define the 'mmap_again' label only for x86_64.

I'm a bit lost. Is the problem that mmap_again is considered to be unused since the compiler has shown all gotos are unreachable?

All ‘goto‘s are inside the ‘#if defined x86_64...‘ so on other platforms none of the ‘goto‘s will be seen by the compiler because the preprocessor removed that code already.

Oct 28 2018, 2:51 PM
trommler added a comment to D5273: Fix unused label on non x86_64 platforms.

Hmm, I tried the same thing in https://phabricator.haskell.org/D5230 but it was causing (very odd) problems.

What kind of problems did you see?

Oct 28 2018, 10:28 AM
trommler created D5273: Fix unused label on non x86_64 platforms.
Oct 28 2018, 10:19 AM

Oct 8 2018

trommler added a comment to D5212: ghc-heap: Fix writing closures on big endian.

I was hoping the fix would also help with Trac #15411 but it doesn't. So I removed the reference to that ticket.

Oct 8 2018, 2:22 AM
trommler updated the Trac tickets for D5212: ghc-heap: Fix writing closures on big endian.
Oct 8 2018, 2:21 AM

Oct 7 2018

trommler created D5212: ghc-heap: Fix writing closures on big endian.
Oct 7 2018, 1:13 PM

Aug 21 2018

trommler added a comment to D5029: Document (however vaguely) the semantics of MO_WriteBarrier.

Perhaps we can avoid negation and say what the guarantee of the barrier actually is.

Aug 21 2018, 12:32 PM

Aug 3 2018

trommler added a comment to D5029: Document (however vaguely) the semantics of MO_WriteBarrier.

Right, this diff was really intended to be a basis for discussion.

My comment was meant as a contribution of an idea to the discussion. Sorry, I should have made that explicit.

Aug 3 2018, 5:08 AM

Aug 1 2018

trommler added a comment to D5029: Document (however vaguely) the semantics of MO_WriteBarrier.

Linux documentation of memory barriers defines a write barrier as follows:

Aug 1 2018, 8:21 AM

Jul 25 2018

Ian Lynagh <ian@well-typed.com> committed rGHCDIFFe3abb5ac8cd0: Propagate ffi includes to compiler/ghci/LibFFI.hsc (Fixes #7686) (authored by trommler).
Propagate ffi includes to compiler/ghci/LibFFI.hsc (Fixes #7686)
Jul 25 2018, 7:59 AM
Ian Lynagh <ian@well-typed.com> committed rGHCDIFF7ee5bedc5e50: Link in-tree libffi to rts. Fixes trac #5743. (authored by trommler).
Link in-tree libffi to rts. Fixes trac #5743.
Jul 25 2018, 7:47 AM

Jul 22 2018

trommler created D5001: Fix endian issues in ghc-heap.
Jul 22 2018, 1:19 PM

Jun 10 2018

trommler updated the diff for D4825: Mark test broken on powerpc64[le].
  • Expand tabs.
Jun 10 2018, 3:50 AM
trommler updated the summary of D4825: Mark test broken on powerpc64[le].
Jun 10 2018, 1:52 AM
trommler created D4825: Mark test broken on powerpc64[le].
Jun 10 2018, 1:47 AM

May 26 2018

trommler added reviewers for D4741: Fix validate for GHCi without TABLES_NEXT_TO_CODE: simonmar, erikd, hvr.
May 26 2018, 4:47 PM
trommler created D4741: Fix validate for GHCi without TABLES_NEXT_TO_CODE.
May 26 2018, 2:35 PM

Mar 9 2018

trommler added a comment to D4489: SPARC nativeGen: Support for MO_SS_Conv_W32_W64.

Note: I don't have a working SPARC machine so I could not test this.

Mar 9 2018, 5:28 AM
trommler created D4489: SPARC nativeGen: Support for MO_SS_Conv_W32_W64.
Mar 9 2018, 5:24 AM

Mar 5 2018

trommler added a comment to D4363: Improve accuracy of get/setAllocationCounter.

Currently MO_SS_Conv W32 W64 is not supported on 32 bit PPC.

I created D4468 for this.

Mar 5 2018, 3:56 AM
trommler created D4468: PPC nativeGen: Add support for MO_SS_Conv_W32_W64.
Mar 5 2018, 3:56 AM

Feb 1 2018

trommler added a comment to D4364: ghc-prim: Emulate C11 atomics when not available.

I think atomic read is fine but atomic write is missing a parameter. See inline comment.

Feb 1 2018, 1:34 PM
trommler added a comment to D4363: Improve accuracy of get/setAllocationCounter.

Currently MO_SS_Conv W32 W64 is not supported on 32 bit PPC. In fact, we would get a compiler panic if I understand the code in nativeGen/PPC/Codegen.hs right.

Feb 1 2018, 12:11 PM

Nov 27 2017

trommler added a comment to D4236: Add new mbmi and mbmi2 compiler flags.

I have a small comment regarding the C functions.

Nov 27 2017, 10:42 AM

Nov 14 2017

trommler accepted D4182: CLabel: Clean up unused label types.

LGTM. I validated this diff applied to HEAD and Phab:D4181 on a POWER8 (powerpc64-linux) with no regressions.

Nov 14 2017, 11:38 AM

Nov 12 2017

trommler requested changes to D4182: CLabel: Clean up unused label types.

I built the diff against HEAD on a POWER8 and got this error:

Nov 12 2017, 1:19 PM
trommler created D4181: RTS: Disable warnings in ffi.h.
Nov 12 2017, 11:19 AM

Nov 6 2017

trommler created D4165: Update autoconf test for gcc to require 4.7 and up.
Nov 6 2017, 1:30 PM

Nov 5 2017

trommler updated the diff for D4149: Fix PPC NCG after blockID patch.
  • Improve comments
Nov 5 2017, 11:46 AM
trommler added inline comments to D4149: Fix PPC NCG after blockID patch.
Nov 5 2017, 2:58 AM
trommler updated the diff for D4149: Fix PPC NCG after blockID patch.
  • Fix duplicate labels on powerpc64le
Nov 5 2017, 2:48 AM

Nov 4 2017

trommler updated the summary of D4149: Fix PPC NCG after blockID patch.
Nov 4 2017, 2:15 PM
trommler updated the summary of D4149: Fix PPC NCG after blockID patch.
Nov 4 2017, 2:15 PM
trommler planned changes to D4149: Fix PPC NCG after blockID patch.

I'd like to fix powerpc64le, too.

Nov 4 2017, 1:10 PM
trommler updated the diff for D4149: Fix PPC NCG after blockID patch.
  • Break up long lines
  • Make block info tables local and simplify
  • Simplify and always use infoTblLbl
  • Add note Proc-point block entry-point
Nov 4 2017, 1:07 PM
trommler updated the summary of D4149: Fix PPC NCG after blockID patch.
Nov 4 2017, 6:17 AM

Nov 3 2017

trommler updated the summary of D4149: Fix PPC NCG after blockID patch.
Nov 3 2017, 2:17 PM
trommler added a reviewer for D4149: Fix PPC NCG after blockID patch: angerman.
Nov 3 2017, 8:47 AM

Nov 2 2017

trommler created D4149: Fix PPC NCG after blockID patch.
Nov 2 2017, 11:44 AM

Oct 2 2017

trommler updated the summary of D4009: Fix atomicread/write operations.
Oct 2 2017, 10:24 AM

Sep 30 2017

trommler updated the diff for D4009: Fix atomicread/write operations.
  • add lost commit
Sep 30 2017, 2:03 PM
trommler updated the diff for D4009: Fix atomicread/write operations.
  • add comment for memory barrier
Sep 30 2017, 1:59 PM

Sep 27 2017

trommler added a comment to D4009: Fix atomicread/write operations.

These get compiled into the same code on x86, right?

For atomic read the answer is yes.

Sep 27 2017, 10:15 AM

Sep 23 2017

trommler added 1 blocking reviewer(s) for D3984: PPC NCG: Impl branch prediction, atomic ops.: hvr.
Sep 23 2017, 9:58 AM
trommler added a comment to D3984: PPC NCG: Impl branch prediction, atomic ops..

I'll have to trust you on the PPC details.

I checked the POWER ISA docs and compared the code with what gcc does. I also added a comment with references to the POWER docs in a place where the code looks very odd.

Sep 23 2017, 9:58 AM
trommler updated the diff for D3984: PPC NCG: Impl branch prediction, atomic ops..
  • add comments and fix typo
Sep 23 2017, 9:58 AM

Sep 22 2017

trommler created D4009: Fix atomicread/write operations.
Sep 22 2017, 12:26 PM

Sep 17 2017

trommler updated the summary of D3984: PPC NCG: Impl branch prediction, atomic ops..
Sep 17 2017, 5:46 AM
trommler created D3984: PPC NCG: Impl branch prediction, atomic ops..
Sep 17 2017, 4:40 AM

May 9 2017

trommler abandoned D3540: RTS: Fix duplicate symbols in unreg'd on ppc64le.

Let's use Ben's patch D3560. It is both cleaner and more efficient.

May 9 2017, 2:14 PM
trommler accepted D3560: rts: Don't build StgCRunAsm.S if unregisterised.

Yes. That is better than my patch.

May 9 2017, 2:12 PM

May 5 2017

trommler created D3540: RTS: Fix duplicate symbols in unreg'd on ppc64le.
May 5 2017, 1:09 PM

Apr 30 2017

trommler created D3512: PPC NCG: Lower MO_*_Fabs as PowerPC fabs instruction.
Apr 30 2017, 4:57 AM

Apr 25 2017

trommler added a comment to D2973: PPC NCG: Implement callish prim ops.
In D2973#99561, @hvr wrote:

Is there any particular test-case that exercises specifically this part of the code-gen? I can't easily run the whole validate testsuite currently, due to a broken Python installation.

Apr 25 2017, 12:28 AM

Mar 27 2017

trommler updated the diff for D2973: PPC NCG: Implement callish prim ops.
  • Fix count trailing zeros formula in comment
Mar 27 2017, 10:08 AM

Mar 25 2017

trommler added a comment to D2973: PPC NCG: Implement callish prim ops.

Fix long lines.
Rebase again to fix Habormaster builds.

Mar 25 2017, 4:35 AM

Mar 24 2017

trommler updated the diff for D2973: PPC NCG: Implement callish prim ops.

Fix long lines.
Rebase again to fix Habormaster builds.

Mar 24 2017, 3:39 AM

Mar 21 2017

trommler updated the diff for D2973: PPC NCG: Implement callish prim ops.

Rebase so it applies cleanly to HEAD again.

Mar 21 2017, 2:21 PM

Jan 17 2017

trommler added a comment to D2973: PPC NCG: Implement callish prim ops.
In D2973#86962, @erikd wrote:

This looks ok to me, but I can't currently built it cause my PowerPC machine is busted.

Jan 17 2017, 4:39 AM
trommler updated D2973: PPC NCG: Implement callish prim ops.
Jan 17 2017, 3:44 AM

Jan 16 2017

trommler updated subscribers of T297: Phabricator stopped sending email.
Jan 16 2017, 4:29 AM
trommler updated subscribers of T297: Phabricator stopped sending email.

When did this start happening?

Jan 16 2017, 3:46 AM

Jan 14 2017

trommler updated the diff for D2973: PPC NCG: Implement callish prim ops.
  • Break up long lines
Jan 14 2017, 11:26 AM
trommler created T297: Phabricator stopped sending email.
Jan 14 2017, 11:12 AM
trommler retitled D2973: PPC NCG: Implement callish prim ops from to PPC NCG: Implement callish prim ops.
Jan 14 2017, 10:09 AM

Dec 18 2016

trommler updated the diff for D2782: Testsuite: Skip failing tests on PowerPC 64-bit.
  • remove newline.
Dec 18 2016, 8:21 AM

Dec 16 2016

trommler updated the diff for D2782: Testsuite: Skip failing tests on PowerPC 64-bit.
  • Fix typos in comment.
  • Improve RTS linker support detection
  • RTS linker: Better error message on PPC64
  • RTS linker: Note runtime-linker-support
Dec 16 2016, 1:39 PM

Dec 6 2016

trommler added a comment to D2791: NCG: Implement trivColorable for PowerPC 64-bit.
In D2791#81436, @hvr wrote:

@trommler just wondering, how does register allocation work right now for ppc64?

Dec 6 2016, 4:19 AM

Dec 5 2016

trommler retitled D2791: NCG: Implement trivColorable for PowerPC 64-bit from to NCG: Implement trivColorable for PowerPC 64-bit.
Dec 5 2016, 12:58 PM
trommler updated the diff for D2782: Testsuite: Skip failing tests on PowerPC 64-bit.
  • Testsuite: rename config property
Dec 5 2016, 9:12 AM

Dec 3 2016

trommler added a comment to D2782: Testsuite: Skip failing tests on PowerPC 64-bit.

I added a new property for runtime linker support to settings

Dec 3 2016, 1:21 PM
trommler updated the diff for D2782: Testsuite: Skip failing tests on PowerPC 64-bit.
  • add to GHC info if RTS linker is supported
  • Testsuite: skip another test when no RTS linker
  • Fix typo.
Dec 3 2016, 1:15 PM

Dec 2 2016

trommler added inline comments to D2782: Testsuite: Skip failing tests on PowerPC 64-bit.
Dec 2 2016, 2:12 AM

Dec 1 2016

trommler retitled D2782: Testsuite: Skip failing tests on PowerPC 64-bit from to Testsuite: Skip failing tests on PowerPC 64-bit.
Dec 1 2016, 1:04 PM

Oct 11 2016

trommler added a comment to D2525: StgCmmPrim: Add missing write barrier..

Inline comment done.

Oct 11 2016, 10:24 AM

Oct 10 2016

trommler updated D2525: StgCmmPrim: Add missing write barrier..
Oct 10 2016, 6:55 AM
trommler updated the diff for D2525: StgCmmPrim: Add missing write barrier..
  • Move write barrier and add comment
Oct 10 2016, 5:16 AM