#8791 closed feature request (fixed)

Use new primitive bitwise operations for Int# in Data.Bits

Reported by: cdk Owned by: cdk
Priority: normal Milestone: 7.8.1
Component: libraries/base Version: 7.6.3
Keywords: Cc:
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: None/Unknown Test Case:
Blocked By: Blocking:
Related Tickets: #7689 Differential Revisions:

Description

New primitive bitwise primops have been added (andI#, orI#, etc). They should be used in the Int instance of Data.Bits to avoid I# (word2Int# (int2Word# ...)) conversions.

Attachments (2)

primitive-bitwise-int-ops.diff (1.5 KB) - added by cdk 14 months ago.
0001-Use-bitwise-primops-for-Int-in-Data.Bits-instances.patch (2.3 KB) - added by cdk 14 months ago.

Download all attachments as: .zip

Change History (7)

Changed 14 months ago by cdk

comment:1 Changed 14 months ago by cdk

  • Status changed from new to patch

comment:2 Changed 14 months ago by cdk

  • Owner set to cdk
  • Summary changed from Use new primitive bitwise operations on Int in Data.Bits to Use new primitive bitwise operations for Int# in Data.Bits

comment:3 Changed 14 months ago by Herbert Valerio Riedel <hvr@…>

In 0fc4fb5477d3ca22a8b6894db5b1112b9badfdc4/base:

Use new bitwise Int# primops in Data.Bits (re #8791)

The new primops (see also #7689) allow to optimize
`instance Bits Int` by allowing to operate directly on Int#
instead of having to convert to Word# and back to Int# again.

Authored-by: Chris Dueck <[email protected]>
Signed-off-by: Herbert Valerio Riedel <[email protected]>

comment:4 Changed 14 months ago by hvr

  • difficulty changed from Unknown to Easy (less than 1 hour)
  • Status changed from patch to merge
  • Type changed from task to feature request

comment:5 Changed 13 months ago by hvr

  • Resolution set to fixed
  • Status changed from merge to closed
Note: See TracTickets for help on using tickets.