Skip to content

Minor code tweaks to work around a compiler bug in MSVC.#30696

Closed
acoates-ms wants to merge 1 commit into
facebook:masterfrom
acoates-ms:msvcfix
Closed

Minor code tweaks to work around a compiler bug in MSVC.#30696
acoates-ms wants to merge 1 commit into
facebook:masterfrom
acoates-ms:msvcfix

Conversation

@acoates-ms
Copy link
Copy Markdown
Contributor

Summary

While pulling in additional react-native code into react-native-windows to support fabric, I hit some bugs with the MSVC compiler. These minor changes work around those bugs and shouldn't have any functional effect on the code

Changelog

[Internal] [Fixed] - Minor code tweaks to work around a compiler bug in MSVC.

Test Plan

Code compiles in MSVC and clang.

@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: Microsoft Partner: Microsoft Partner labels Jan 6, 2021
@analysis-bot
Copy link
Copy Markdown

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

Base commit: d3a3ce8

@analysis-bot
Copy link
Copy Markdown

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 8,884,740 +101
android hermes armeabi-v7a 8,383,151 +117
android hermes x86 9,373,055 +44
android hermes x86_64 9,318,157 +112
android jsc arm64-v8a 10,340,085 +102
android jsc armeabi-v7a 9,821,605 +127
android jsc x86 10,389,907 +49
android jsc x86_64 10,975,263 +117

Base commit: d3a3ce8

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@appden has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@appden merged this pull request in 47bb3be.

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Jan 8, 2021
@acoates-ms acoates-ms deleted the msvcfix branch September 2, 2022 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. p: Microsoft Partner: Microsoft Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants