Skip to content

multiple support version go1.16 go1.17 go1.18#1

Open
visualfc wants to merge 18 commits intomasterfrom
multiver
Open

multiple support version go1.16 go1.17 go1.18#1
visualfc wants to merge 18 commits intomasterfrom
multiver

Conversation

@visualfc
Copy link
Copy Markdown
Owner

@visualfc visualfc commented Oct 4, 2022

No description provided.

visualfc and others added 13 commits September 23, 2022 15:52
compiler/prelude: fix array type size
…pherjs#1157)

* compiler/natives/src/reflect: fix valueIntrface check struct copy

Fixed gopherjs#1156
compiler: support `go:linkname` directive for methods

This is similar to what the upstream compiler supports. This functionality is inherently unsafe, but can be useful for some certain libraries like [reflectx](https://github.com/goplusjs/reflectx/blob/main/name_js.go#L17). The first argument of the function will act as a receiver of the linked method.

As long as underlying typed for the first arguments match, they will be converted at runtime, which allows linking to methods of unexported types. However, types of the other arguments are not converted, nor the signature is verified to match the linked method.
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 9, 2022

Reference app: jQuery TodoMVC (acf500a6c32a83d8c4582d967b09a65febf0e120)

BRANCH ORIGINAL MINIFIED COMPRESSED (GZIP)
Pull request (multiver) 2,883,396 bytes 1,898,969 bytes 381,513 bytes
Target branch (master) 0.05% increase (1,382 bytes) 0.04% increase (747 bytes) 0.04% increase (156 bytes)

#outputSize

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants