tshort / StaticCompiler.jl

Compiles Julia code to a standalone library (experimental)
Other
488 stars 31 forks source link

Update for GPUCompiler v0.21 #136

Closed tshort closed 1 year ago

tshort commented 1 year ago

Upgrading GPUCompiler didn't fix failures on macOS ☹️. Nightlies are all still failing, too.

brenhinkeller commented 1 year ago

The failures in parsedlm are something that needs to be fixed over in statictools (used to be only on aarch64 but maybe on all apple now?) but I think we can just label those as broken, let's see how much closer this gets us...

MasonProtter commented 1 year ago

I don't know if this is related, but @device_override is completely broken and I'm not sure if it ever worked. I can't find a commit right now where it worked at least.

julia> using StaticCompiler, StaticTools

julia> f() = printf(c"the value is: %d\n", 1);

julia> StaticCompiler.@device_override f() = printf(c"the value is: %d\n", 2);

julia> compile_executable(f, (), "./")
"/home/mason/f"

julia> run(`./f`) # run it
the value is: 1
Process(`./f`, ProcessExited(0))
brenhinkeller commented 1 year ago

Hmm, could that be some weird world age issue or something? The versions defined in https://github.com/tshort/StaticCompiler.jl/blob/master/src/quirks.jl seem to be doing something because without them https://github.com/tshort/StaticCompiler.jl/blob/master/test/scripts/throw_errors.jl wouldn't compile, IIRC

brenhinkeller commented 1 year ago

Ok, looks like we're back to being fine except for Nightly now

tshort commented 1 year ago

Should we stop testing nightly now, too?

It looks like it's hitting an error in optimize!, so something must have changed there.

brenhinkeller commented 1 year ago

I'd leave the nightly tests since it's always good to know what's coming --- they show up in a different badge on the repo anyways so the main two badges should be green after merging this. The red nightly can just be an encouragement to go raise the issue on Slack if things stay broken there and a release is coming...

tshort commented 1 year ago

Ahh. Good to know.