codereport / jsource

J Language Source Code. Livestream links ⬇️
https://www.youtube.com/playlist?list=PLVFrD1dmDdvfVhYLU_iKkV67X9XqCJLWe
Other
38 stars 20 forks source link

Remove blis, gemm etc. #69

Closed juntuu closed 3 years ago

juntuu commented 3 years ago

Removed blis and gemm which rendered cpuinfo basically unused (except for two undocumented foreign functions, which are now removed), resolving part of #13.

Also removed the few remaining PREFETCHes (resolves #61)

codereport commented 3 years ago

@juntuu Are you running on osx or linux? Last time we removed blis we bricked osx. If linux, @ahmadbeirkdar are you able to run locally on your mac to see if this branch is good?

ahmadbeirkdar commented 3 years ago

@juntuu Are you running on osx or linux? Last time we removed blis we bricked osx. If linux, @ahmadbeirkdar are you able to run locally on your mac to see if this branch is good?

👍 Tested it worked!

 ~/De/Projects/jsource │ removal ?73  ninja -C build                                                   ✔ │ 3s │ 08:17:59 pm 
ninja: Entering directory `build'
[0/1] Re-running CMake...
-- Configuring done
-- Generating done
-- Build files have been written to: /Users/ahmadbeirkdar/Desktop/Projects/jsource/build
[123/123] Linking C shared library jsrc/Debug/libj.dylib
99% tests passed, 1 tests failed out of 396

Total Test time (real) =  65.15 sec

The following tests FAILED:
    179 - g420stch (SEGFAULT)
Errors while running CTest