delta/go-git.git/src/internal/reflectlite, branch master

internal/reflectlite: common up types, remove code

2023-05-11T13:45:58+00:00

Change-Id: I4d3ef025b935e52c47896f69814ba2f1a504d749
Reviewed-on: https://go-review.googlesource.com/c/go/+/489375
Reviewed-by: Keith Randall 
Reviewed-by: Keith Randall 
TryBot-Result: Gopher Robot 
Run-TryBot: David Chase

reflect: prepare fieldnames for moving to abi/type.go

2023-05-10T22:55:30+00:00

Change-Id: Ia8e88029d29a1210dc7a321578e61336e235f35a
Reviewed-on: https://go-review.googlesource.com/c/go/+/487555
Reviewed-by: Keith Randall 
TryBot-Result: Gopher Robot 
Run-TryBot: David Chase 
Reviewed-by: Keith Randall

internal/abi,reflectlite,reflect,runtime: common up chan type

2023-05-10T22:45:24+00:00

Change-Id: I085b61c544b85d70fabb1c0d9fe91207826dd21a
Reviewed-on: https://go-review.googlesource.com/c/go/+/484858
TryBot-Result: Gopher Robot 
Reviewed-by: Keith Randall 
Run-TryBot: David Chase 
Reviewed-by: Keith Randall

internal/reflectlite, runtime: move more constants and types into internal/abi

2023-05-10T22:43:30+00:00

Change-Id: If5da1057ead34eb3e4c7f42bbe6ad3d350b97725
Reviewed-on: https://go-review.googlesource.com/c/go/+/484856
Reviewed-by: Keith Randall 
Reviewed-by: Keith Randall 
Run-TryBot: David Chase 
TryBot-Result: Gopher Robot

internal/abi: common up ArrayType

2023-05-10T21:44:37+00:00

This refactoring is more problematic because the client
package wrap abi.Type, thus the self-referential fields
within ArrayType need to be downcast to the client wrappers
in several places.  It's not clear to me this is worthwhile;
this CL is for additional comment, before I attempt similar
changes for other self-referential types.

Change-Id: I41e517e6d851b32560c41676b91b76d7eb17c951
Reviewed-on: https://go-review.googlesource.com/c/go/+/466236
Run-TryBot: David Chase 
Reviewed-by: Keith Randall 
TryBot-Result: Gopher Robot 
Reviewed-by: Keith Randall

internal/abi: common up Method, Imethod, UncommonType types

2023-05-10T21:41:14+00:00

was two commits, the first contained a lot of intermediate work,
better this way.

Change-Id: I7c5b79ef78b21a85828c8aaf9baeae86bb144db7
Reviewed-on: https://go-review.googlesource.com/c/go/+/463118
Reviewed-by: Keith Randall 
Run-TryBot: David Chase 
Reviewed-by: Keith Randall 
TryBot-Result: Gopher Robot

internal/abi: refactor (basic) type struct into one definition

2023-05-05T14:59:28+00:00

This touches a lot of files, which is bad, but it is also good,
since there's N copies of this information commoned into 1.

The new files in internal/abi are copied from the end of the stack;
ultimately this will all end up being used.

Change-Id: Ia252c0055aaa72ca569411ef9f9e96e3d610889e
Reviewed-on: https://go-review.googlesource.com/c/go/+/462995
TryBot-Result: Gopher Robot 
Reviewed-by: Carlos Amedee 
Run-TryBot: David Chase 
Reviewed-by: Keith Randall

internal/reflectlite: use unsafe.String in name.name and name.tag

2022-11-10T18:42:48+00:00

Same as CL 448675 did in package reflect.

Change-Id: I26277d8dcf2d2e204724d6fa5cc6e1ad391633f5
Reviewed-on: https://go-review.googlesource.com/c/go/+/448936
Reviewed-by: Michael Knyszek 
Run-TryBot: Ian Lance Taylor 
Auto-Submit: Tobias Klauser 
TryBot-Result: Gopher Robot 
Run-TryBot: Tobias Klauser 
Reviewed-by: Ian Lance Taylor

cmd/compile,runtime,reflect: move embedded bit from offset to name

2022-06-14T23:22:11+00:00

Previously we stole a bit from the field offset to encode whether
a struct field was embedded.

Instead, encode that bit in the name field, where we already have
some unused bits to play with. The bit associates naturally with
the name in any case.

This leaves a full uintptr to specify field offsets. This will make
the fix for #52740 cleaner.

Change-Id: I0bfb85564dc26e8c18101bc8b432f332176d7836
Reviewed-on: https://go-review.googlesource.com/c/go/+/412138
Reviewed-by: Cherry Mui 
Run-TryBot: Keith Randall 
TryBot-Result: Gopher Robot 
Reviewed-by: Keith Randall

cmd/compile: set LocalPkg.Path to -p flag

2022-05-16T18:19:47+00:00

Since CL 391014, cmd/compile now requires the -p flag to be set the
build system. This CL changes it to initialize LocalPkg.Path to the
provided path, rather than relying on writing out `"".` into object
files and expecting cmd/link to substitute them.

However, this actually involved a rather long tail of fixes. Many have
already been submitted, but a few notable ones that have to land
simultaneously with changing LocalPkg:

1. When compiling package runtime, there are really two "runtime"
packages: types.LocalPkg (the source package itself) and
ir.Pkgs.Runtime (the compiler's internal representation, for synthetic
references). Previously, these ended up creating separate link
symbols (`"".xxx` and `runtime.xxx`, respectively), but now they both
end up as `runtime.xxx`, which causes lsym collisions (notably
inittask and funcsyms).

2. test/codegen tests need to be updated to expect symbols to be named
`command-line-arguments.xxx` rather than `"".foo`.

3. The issue20014 test case is sensitive to the sort order of field
tracking symbols. In particular, the local package now sorts to its
natural place in the list, rather than to the front.

Thanks to David Chase for helping track down all of the fixes needed
for this CL.

Updates #51734.

Change-Id: Iba3041cf7ad967d18c6e17922fa06ba11798b565
Reviewed-on: https://go-review.googlesource.com/c/go/+/393715
Reviewed-by: David Chase 
Run-TryBot: Matthew Dempsky 
TryBot-Result: Gopher Robot 
Reviewed-by: Cuong Manh Le