From f89d24dd5a760e07c820fadcd7fae8f275a4aef5 Mon Sep 17 00:00:00 2001
From: KristofferC <kcarlsson89@gmail.com>
Date: Tue, 5 May 2020 17:18:44 +0200
Subject: [PATCH] Revert "manually destruct and reconstruct subarray when
 throwing boundserror (#29867)"

This reverts commit 13694f4210f42a8e7a0d282296422b1d0a07b0f4.
---
 base/subarray.jl | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/base/subarray.jl b/base/subarray.jl
index a540ef1811e36..1df45f8d78781 100644
--- a/base/subarray.jl
+++ b/base/subarray.jl
@@ -43,14 +43,6 @@ check_parent_index_match(parent::AbstractArray{T,N}, ::NTuple{N, Bool}) where {T
 check_parent_index_match(parent, ::NTuple{N, Bool}) where {N} =
     throw(ArgumentError("number of indices ($N) must match the parent dimensionality ($(ndims(parent)))"))
 
-# This makes it possible to elide view allocation in cases where the
-# view is indexed with a boundscheck but otherwise all its uses
-# are inlined
-@inline Base.throw_boundserror(A::SubArray, I) =
-    __subarray_throw_boundserror(typeof(A), A.parent, A.indices, A.offset1, A.stride1, I)
-@noinline __subarray_throw_boundserror(::Type{T}, parent, indices, offset1, stride1, I) where {T} =
-    throw(BoundsError(T(parent, indices, offset1, stride1), I))
-
 # This computes the linear indexing compatibility for a given tuple of indices
 viewindexing(I::Tuple{}) = IndexLinear()
 # Leading scalar indices simply increase the stride