Fix strides for Adjoint/Transpose of vectors

The previous method took the first stride of the parent vector and just
doubled it. This copies what base does.

Author: Tokazama

src/stridelayout.jl

@@ -418,13 +418,10 @@ end
 @inline strides(A::Array{<:Any,N}) where {N} = (StaticInt(1), Base.tail(Base.strides(A))...)
 @inline strides(A::AbstractArray) = _strides(A, Base.strides(A), contiguous_axis(A))
 
-@inline function strides(x::LinearAlgebra.Adjoint{T,V}) where {T,V<:AbstractVector{T}}
-    strd = stride(parent(x), One())
-    return (strd, strd)
-end
-@inline function strides(x::LinearAlgebra.Transpose{T,V}) where {T,V<:AbstractVector{T}}
-    strd = stride(parent(x), One())
-    return (strd, strd)
+function strides(x::VecAdjTrans)
+    p = parent(x)
+    st = first(strides(p))
+    return (static_length(p) * st, st)
 end
 
 @generated function _strides(A::AbstractArray{T,N}, s::NTuple{N}, ::StaticInt{C}) where {T,N,C}

test/runtests.jl

@@ -480,10 +480,13 @@ end
     @test @inferred(ArrayInterface.strides(S)) === (StaticInt(1), StaticInt(2), StaticInt(6))
     @test @inferred(ArrayInterface.strides(Sp)) === (StaticInt(6), StaticInt(1), StaticInt(2))
     @test @inferred(ArrayInterface.strides(Sp2)) === (StaticInt(6), StaticInt(2), StaticInt(1))
+
+    @test @inferred(ArrayInterface.strides(view(Sp2, :, 1, 1)')) === (12, StaticInt(6))
+
     @test @inferred(ArrayInterface.stride(Sp2, StaticInt(1))) === StaticInt(6)
     @test @inferred(ArrayInterface.stride(Sp2, StaticInt(2))) === StaticInt(2)
     @test @inferred(ArrayInterface.stride(Sp2, StaticInt(3))) === StaticInt(1)
-    
+
     @test @inferred(ArrayInterface.strides(M)) === (StaticInt(1), StaticInt(2), StaticInt(6))
     @test @inferred(ArrayInterface.strides(Mp)) === (StaticInt(2), StaticInt(6))
     @test @inferred(ArrayInterface.strides(Mp2)) === (StaticInt(1), StaticInt(6))