Don't export MTL

[christiangnrd]

As discussed in #359 (comment).

TODO:

• Maybe remove Unexport CPUStorage from MTL?
• Fix storage mode output of MtlArrays

[christiangnrd]

The scripts test isn't failing on my machine and I don't know if it's related.

[maleadt]

The scripts test isn't failing on my machine and I don't know if it's related.

ERROR: LoadError: UndefVarError: air_support not defined

That seems related?

[maleadt]

The scripts test isn't failing on my machine and I don't know if it's related.

You realize the tests don't run by default?

Metal.jl/test/scripts.jl

Lines 7 to 9 in 116b1bf

	if get(ENV, "BUILDKITE_PIPELINE_NAME", "") != "Metal.jl"
	@warn "Skipping script tests"

I'll push a fix.

[christiangnrd]

I didn't when I wrote that comment but noticed this morning when I started looking into it. I turned the errors into failures and was going to look into it more later in the day.

[maleadt]

I'm not sure the preference needs to be renamed. storage = PrivateStorage seems redundant? Maybe we should even use simpler, lower-case identifiers instead, like in CUDA.jl (private, managed, shared).

[christiangnrd]

This should be ready to merge after final review.