Skip to content

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

Closed
christophpurrer wants to merge 1 commit into
facebook:mainfrom
christophpurrer:export-D41482821
Closed

Enable C++17 for cxxreact and jsiexecutor/inspector and ...#35435
christophpurrer wants to merge 1 commit into
facebook:mainfrom
christophpurrer:export-D41482821

Conversation

@christophpurrer
Copy link
Copy Markdown
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
Copy Markdown
Contributor

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

@analysis-bot
Copy link
Copy Markdown

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
Copy Markdown

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

Base commit: 6410a62
Branch: main

@pull-bot
Copy link
Copy Markdown

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
Copy Markdown
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
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