Skip to content
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

Enable C++17 for cxxreact and jsiexecutor/inspector and ... #35435

Closed

Conversation

christophpurrer
Copy link
Contributor

Summary:
This raises the C++ language standard to C++17 which is needed to remove some folly:: dependencies

changelog: [Internal]

Differential Revision: D41482821

Summary:
This raises the C++ language standard to C++17 which is needed to remove some folly:: dependencies

changelog: [Internal]

Differential Revision: D41482821

fbshipit-source-id: e39c68948b36430338af5a2d8a52f1988e483881
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner fb-exported labels Nov 22, 2022
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D41482821

@analysis-bot
Copy link

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 7,103,605 +1,017
android hermes armeabi-v7a 6,471,923 +1,254
android hermes x86 7,521,235 +1,170
android hermes x86_64 7,380,107 +1,357
android jsc arm64-v8a 8,968,472 +1,018
android jsc armeabi-v7a 7,699,584 +1,264
android jsc x86 9,030,768 +1,165
android jsc x86_64 9,508,796 +1,343

Base commit: 0d3d002
Branch: main

@analysis-bot
Copy link

Platform Engine Arch Size (bytes) Diff
ios - universal n/a --

Base commit: 6410a62
Branch: main

@pull-bot
Copy link

PR build artifact for 1e62463 is ready.
To use, download tarball from "Artifacts" tab in this CircleCI job then run yarn add <path to tarball> in your React Native project.

@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @christophpurrer in 9517320.

When will my fix make it into a release? | Upcoming Releases

@react-native-bot react-native-bot added the Merged This PR has been merged. label Nov 24, 2022
OlimpiaZurek pushed a commit to OlimpiaZurek/react-native that referenced this pull request May 22, 2023
…#35435)

Summary:
Pull Request resolved: facebook#35435

This raises the C++ language standard to C++17 which is needed to remove some folly:: dependencies

changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D41482821

fbshipit-source-id: 62af6e95e6e78378112a4ce4e9c3ab7df0587218
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants