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

Collapse write barrier function for HV and SHV #1501

Open
wants to merge 8 commits into
base: static_h
Choose a base branch
from

Conversation

lavenzg
Copy link
Contributor

@lavenzg lavenzg commented Sep 6, 2024

Summary:
Collapse them to GCHermesValueBase since the implementation is
exactly the same. This will make future changes to these functions
easier.

Differential Revision: D62158417

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Sep 6, 2024
@facebook-github-bot
Copy link
Contributor

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

@lavenzg lavenzg force-pushed the export-D62158417-to-static_h branch from 582f66a to 312fcf8 Compare October 22, 2024 00:08
lavenzg added a commit to lavenzg/hermes that referenced this pull request Oct 22, 2024
Summary:

Collapse them to `GCHermesValueBase` since the implementation is
exactly the same. This will make future changes to these functions
easier.

Differential Revision: D62158417
@facebook-github-bot
Copy link
Contributor

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

Differential Revision: D61675022
Differential Revision: D61747499
Differential Revision: D61807366
Differential Revision: D64713261
Summary:
Pull Request resolved: facebook#1501

Collapse them to `GCHermesValueBase` since the implementation is
exactly the same. This will make future changes to these functions
easier.

Differential Revision: D62158417
@facebook-github-bot
Copy link
Contributor

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

@lavenzg lavenzg force-pushed the export-D62158417-to-static_h branch from 312fcf8 to b3db0ce Compare November 8, 2024 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants