Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Conversation

visualfc
Copy link
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

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