m-j-w / CpuId.jl

Ask the CPU for cache sizes, SIMD feature support, a running hypervisor, and more.
Other
54 stars 10 forks source link

Warning on Julia v1.0 #32

Closed juliohm closed 5 years ago

juliohm commented 5 years ago

This warning is quite mysterious but is always triggered when a package is being precompiled that depends on CpuId.jl:

WARNING: eval from module CpuId to ImageQuilting:    
Expr(:call, :cpucycle) = Expr(:block, #= Symbol("/home/juliohm/.julia/packages/CpuId/LfVXU/src/CpuId.jl"):925 =#, Expr(:call, :rdtsc))
  ** incremental compilation may be broken for this module **

WARNING: eval from module CpuId to ImageQuilting:    
Expr(:call, :cpucycle_id) = Expr(:block, #= Symbol("/home/juliohm/.julia/packages/CpuId/LfVXU/src/CpuId.jl"):930 =#, Expr(:call, :rdtscp))
  ** incremental compilation may be broken for this module **

Could you please help fix it?

juliohm commented 5 years ago

@m-j-w could you please take a look at @fredrikekre proposed changes?

m-j-w commented 5 years ago

Sure, apart from simply merging it, do you have a minimum working example that provokes the above warning message? I've not seen the warning myself so far.

juliohm commented 5 years ago

Simply doing using ImageQuilting will cause the warning during precompilation. After it is precompiled, the warning is gone.

m-j-w commented 5 years ago

Ok, it seems simply precompiling a package that includes it already triggers the warning.

juliohm commented 5 years ago

@m-j-w I appreciate if you can tag a new release for this. :+1: