Crazycolorz5 (Adam Chen)
User

Projects

User does not belong to any projects.

User Details

User Since
Fri, Jun 22, 10:20 PM (4 w, 6 h)

Recent Activity

Fri, Jul 13

Crazycolorz5 updated the summary of D4889: rts: address trac issue #13165; Specialize hashing at call site rather than in struct..
Fri, Jul 13, 8:53 PM
Crazycolorz5 added a comment to D4889: rts: address trac issue #13165; Specialize hashing at call site rather than in struct..

Sorry for the delay. Aside from additional benchmarks (which I'll upload soon, after ghc builds), is there any additional work needed on this?

Fri, Jul 13, 8:16 PM

Fri, Jul 6

Crazycolorz5 added a comment to D4889: rts: address trac issue #13165; Specialize hashing at call site rather than in struct..

What were you trying to measure here? This is a dynamically linked binary and so doesn't include the code of the RTS itself.

Fri, Jul 6, 8:07 PM

Mon, Jul 2

Crazycolorz5 updated the diff for D4889: rts: address trac issue #13165; Specialize hashing at call site rather than in struct..
  • Fixed expand always using hashWord.
Mon, Jul 2, 11:41 PM
Crazycolorz5 updated the diff for D4889: rts: address trac issue #13165; Specialize hashing at call site rather than in struct..
  • Fixed expand always using hashWord.
Mon, Jul 2, 11:17 PM

Fri, Jun 29

Crazycolorz5 added a comment to D4889: rts: address trac issue #13165; Specialize hashing at call site rather than in struct..

I'm unsure why the tests are failing. The logs seem to indicate library loading issues, but I haven't touched anything in the process of compilation to Core?

Fri, Jun 29, 9:09 AM

Thu, Jun 28

Crazycolorz5 updated the diff for D4889: rts: address trac issue #13165; Specialize hashing at call site rather than in struct..
  • Lint fix changes.
Thu, Jun 28, 8:36 PM
Crazycolorz5 retitled D4889: rts: address trac issue #13165; Specialize hashing at call site rather than in struct. from rts: address trac issue #13165. to rts: address trac issue #13165; Specialize hashing at call site rather than in struct..
Thu, Jun 28, 8:22 PM
Crazycolorz5 added a comment to D4889: rts: address trac issue #13165; Specialize hashing at call site rather than in struct..

Seems like a sensible idea. What is the effect on code size?

Some nits:

  • Please update the diff title to say what the change is.
  • Please add the benchmark results (or a link to them) to the summary
  • Please fix the long lines (we have an 80-column policy in the RTS)
Thu, Jun 28, 8:09 PM

Sun, Jun 24

Crazycolorz5 added a comment to D4889: rts: address trac issue #13165; Specialize hashing at call site rather than in struct..

Seems like a sensible thing to do.

Have you characterised how much of an effect this has on real hashing performance?

Sun, Jun 24, 5:58 PM
Crazycolorz5 updated the diff for D4889: rts: address trac issue #13165; Specialize hashing at call site rather than in struct..
  • Removed inline in header due to reference of implementation of HashTable.
Sun, Jun 24, 5:31 PM
Herald added a reviewer for D4889: rts: address trac issue #13165; Specialize hashing at call site rather than in struct.: erikd.
Sun, Jun 24, 1:05 AM