Open rbergen opened 8 months ago
I don't know why I was credited in this code. It could be that the author asked for some help on the Discord back then. Anyway, I had a look and it requires substantial changes to get this to run:
import
statements within a when
. The way to do this is to add a foo_windows.odin
and foo_linux.odin
file that take care of those platform imports depending on the platform. More ideally a get_cpu_topology
kind of procedure would be added to core:sys/info
, which would tell you how many cores and threads, and if they're E or P cores.-opt:3
has to be be replaced with -o:speed
(or -o:aggressive
if using LLVM 17)Semaphore
became Sema
. There's more primitives as well that could be looked at.And beyond that there may be more. Might be worth someone doing a from scratch rewrite if they're interested.
@Kelimion Thanks for looking into this to this level, that's very helpful for whoever chooses to pick up making the required changes.
While updating a number of Docker images published on Docker Hub in the context of this project, I found that the Odin solution does not compile on the current version of Odin.
Without wanting to imply an obligation on the original contributors of the solution, I'm calling out to @omgitsmoe, @gingerBill and @Kelimion in case they would be willing and able to look into this.