mirror of
https://github.com/hanzoai/visor.git
synced 2026-09-24 17:04:14 +00:00
Michael PrattandgVisor bot
f941bf69c4
All remaining uses of gohacks.SliceHeader are to create a slice pointing to some other backing array. Go 1.20 introduces unsafe.Slice to do exactly this, so switch to this interface. For now we want code to continue to build with Go 1.19, so we still use a gohacks.Slice wrapper, which uses unsafe.Slice on 1.20 and SliceHeader on <1.20. Once 1.19 support is dropped, uses can drop gohacks altogether. gohacks.Slice is inlined into callers, and unsafe.Slice is a compiler intrinsic, so these wrappers have minimal overhead. The primary difference is the addition of a nil check on the pointer and an overflow check on ptr+length*size. I think these are minimal enough to not cause problems, but if they are (e.g., in safemem), we could consider adding a SliceUnchecked function that continues to use SliceHeader. But I'd like to try to avoid that, as it adds process to verify it is still compatible with new Go releases. For #8422. PiperOrigin-RevId: 504926819
31 lines
936 B
Go
31 lines
936 B
Go
// Copyright 2023 The gVisor Authors.
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
//go:build go1.20
|
|
|
|
package gohacks
|
|
|
|
import (
|
|
"unsafe"
|
|
)
|
|
|
|
// Slice returns a slice whose underlying array starts at ptr an which length
|
|
// and capacity are len.
|
|
//
|
|
// Slice is a wrapper around unsafe.Slice. Prefer to use unsafe.Slice directly
|
|
// if possible.
|
|
func Slice[T any](ptr *T, length int) []T {
|
|
return unsafe.Slice(ptr, length)
|
|
}
|