Merge pull request #530 from JuliaGPU/tb/copy_ctors

Clean-up copy constructors.

Author: maleadt

src/array.jl

@@ -172,14 +172,11 @@ Base.pointer(x::CuArray, i::Integer) = x.ptr + (i-1) * Base.elsize(x)
 
 @inline function CuArray{T,N}(xs::AbstractArray{T,N}) where {T,N}
   A = CuArray{T,N}(undef, size(xs))
-  if isbits(xs)
-    A .= xs
-  else
-    copyto!(A, collect(xs))
-  end
+  copyto!(A, xs)
   return A
 end
 
+# FIXME: `map(T, xs)`, https://github.com/FluxML/Flux.jl/issues/958
 CuArray{T,N}(xs::AbstractArray{S,N}) where {T,N,S} = CuArray{T,N}((x -> T(x)).(xs))
 
 # underspecified constructors