Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

llvm-native: accelerate-nofib wrongly reports errors #182

Closed
tvh opened this issue Jul 14, 2014 · 1 comment
Closed

llvm-native: accelerate-nofib wrongly reports errors #182

tvh opened this issue Jul 14, 2014 · 1 comment
Labels
llvm backend accelerate-llvm

Comments

@tvh
Copy link
Contributor

tvh commented Jul 14, 2014

When I run accelerate-nofib with +RTS -N4, It reports errors where there aren't any.

        square: [Failed]
*** Failed! Falsifiable (after 8 tests and 2 shrinks): 
Array (Z :. 1 :. 3) [1,-2,1]
*** Expected:
Array (Z :. 1 :. 3) [1,4,1]
*** Received:
Array (Z :. 1 :. 3) [1,4,1]
@tmcdonell
Copy link
Member

Okay, so run itself is not thread-safe (which is annoying, but also why there is runWith), and we weren't correctly telling test-framework to only run a single test at a time for the LLVM CPU backend. The latter at least is fixed.

tmcdonell added a commit to tmcdonell/accelerate-llvm that referenced this issue Aug 24, 2016
We can't call this concurrently from different threads, but add instructions for how to do this instead.

Initially picked up by AccelerateHS/accelerate#182
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
llvm backend accelerate-llvm
Projects
None yet
Development

No branches or pull requests

2 participants