Install fails on AMD 8350 about loopvectorization.jl HOT 5 CLOSED

Just for reference, Wikipedia has a pretty good list of specs for FX processors: https://en.m.wikipedia.org/wiki/List_of_AMD_FX_microprocessors

from loopvectorization.jl.

I no longer call how I decided not to use:

using CpuId
cache_size = CpuId.cachesize()
num_cores = CpuId.cpucores()

https://github.com/chriselrod/VectorizationBase.jl/blob/master/deps/build.jl#L10

For AMD CPUS. Mind trying them and reporting what they return?
It would be better if we don't have to special case.

If that doesn't work, the problem is in CpuId.jl. If it does work, I should probably drop the checks.
Either way, the issue can be addressed in VectorizationBase, like @simeonschaub noted on discourse.

Otherwise, I'll add them. I also need to add Sys.CPU_NAME == "znver2".
What is your Sys.CPU_NAME?

from loopvectorization.jl.

julia> cache_size = CpuId.cachesize()
(16384, 2097152, 8388608)

julia> num_cores = CpuId.cpucores()
8

julia> Sys.CPU_NAME
"bdver1"

from loopvectorization.jl.

Looks like I may be able to just drop the special casing for AMD.

Do you know how I could test on zenv1 and zenv2?
I'm wondering if I should still special case these, or if they're properly supported. It isn't immediately obvious to me how to find out without access to them.

EDIT:
m-j-w/CpuId.jl@2ac67b0
Looks like as of July 2018, those functions should be supported on AMD.

I must have written the build script before that.

I'll just drop the special casing.

from loopvectorization.jl.

Fixed by
JuliaSIMD/VectorizationBase.jl@720d707

Update to VectorizationBase v0.1.8. I'll release a new LoopVectorization with that as the lower bound in the next few days.

from loopvectorization.jl.