Skip to content

[Wasm][Runtime] Interpret absolute function pointer in runtime structures #42095

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

Conversation

kateinoigakukun
Copy link
Member

When SWIFT_COMPACT_ABSOLUTE_FUNCTION_POINTER is enabled, relative direct
pointers whose pointees are functions will be turned into absolute
pointer at compile-time.

Based on #42094

@kateinoigakukun
Copy link
Member Author

@swift-ci please test

@kateinoigakukun kateinoigakukun force-pushed the pr-26594e2245d681b594f6e1b190c8107a1c6dbb1d branch from 00fc9c5 to 934dcbd Compare March 31, 2022 15:03
@kateinoigakukun
Copy link
Member Author

@swift-ci Please test

@kateinoigakukun kateinoigakukun force-pushed the pr-26594e2245d681b594f6e1b190c8107a1c6dbb1d branch from 934dcbd to e9cf573 Compare April 1, 2022 03:47
…ures

When SWIFT_COMPACT_ABSOLUTE_FUNCTION_POINTER is enabled, relative direct
pointers whose pointees are functions will be turned into absolute
pointer at compile-time.
@kateinoigakukun kateinoigakukun force-pushed the pr-26594e2245d681b594f6e1b190c8107a1c6dbb1d branch from e9cf573 to 8c598e9 Compare April 2, 2022 04:32
@kateinoigakukun
Copy link
Member Author

@swift-ci Please test Windows platform

1 similar comment
@kateinoigakukun
Copy link
Member Author

@swift-ci Please test Windows platform

@kateinoigakukun
Copy link
Member Author

@swift-ci Please test

@kateinoigakukun
Copy link
Member Author

The failing Windows CI seems unrelated.

Failed Tests (1):
  Swift(windows-x86_64) :: Driver/enable_regex_literals_flag.swift

@kateinoigakukun kateinoigakukun merged commit f492250 into swiftlang:main Apr 2, 2022
@kateinoigakukun kateinoigakukun deleted the pr-26594e2245d681b594f6e1b190c8107a1c6dbb1d branch April 2, 2022 11:04
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.

1 participant