AccelerateHS / accelerate-llvm

LLVM backend for Accelerate
http://www.acceleratehs.org
Other
155 stars 50 forks source link

Support bytestring-0.11.3.0 #95

Closed tomsmeding closed 1 year ago

tomsmeding commented 1 year ago

Description

Uses CPP to remove a polyfill for takeWhile on ShortByteString, that was necessary before bytestring-0.11.3.0 but is now provided by the source package.

Motivation and context

Compatibility.

How has this been tested?

It builds here :)

Types of changes

Checklist:

dpvanbalen commented 1 year ago

As far as I can see, the same Bytestring update also added "take", right?

tomsmeding commented 1 year ago

@dpvanbalen Thanks for having a look!

After personal communication, I updated the diff to what we agreed. Looks good?