Skip to content

Conversation

@dbort
Copy link
Contributor

@dbort dbort commented Jan 25, 2024

Summary:
This new version:

  • Can fail non-fatally
  • Uses RAII to clean up the memory
  • Uses separate files for lean and aten modes
  • Has tests
  • Lives somewhere other than util.h

Differential Revision: D52645309

@pytorch-bot
Copy link

pytorch-bot bot commented Jan 25, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/1712

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit b08d897 with merge base 23aceb8 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jan 25, 2024
@facebook-github-bot
Copy link
Contributor

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

dbort added a commit to dbort/executorch that referenced this pull request Jan 26, 2024
Summary:

This new version:
- Can fail non-fatally
- Uses RAII to clean up the memory
- Uses separate files for lean and aten modes
- Has tests
- Lives somewhere other than util.h

Reviewed By: larryliu0820

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

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

dbort added a commit to dbort/executorch that referenced this pull request Jan 26, 2024
Summary:

This new version:
- Can fail non-fatally
- Uses RAII to clean up the memory
- Uses separate files for lean and aten modes
- Has tests
- Lives somewhere other than util.h

Reviewed By: larryliu0820

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

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

Summary:

This new version:
- Can fail non-fatally
- Uses RAII to clean up the memory
- Uses separate files for lean and aten modes
- Has tests
- Lives somewhere other than util.h

Reviewed By: larryliu0820

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

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

dbort added a commit to dbort/executorch that referenced this pull request Jan 26, 2024
Summary:

This new version:
- Can fail non-fatally
- Uses RAII to clean up the memory
- Uses separate files for lean and aten modes
- Has tests
- Lives somewhere other than util.h

Reviewed By: larryliu0820

Differential Revision: D52645309
dbort added a commit to dbort/executorch that referenced this pull request Jan 26, 2024
Summary:

This new version:
- Can fail non-fatally
- Uses RAII to clean up the memory
- Uses separate files for lean and aten modes
- Has tests
- Lives somewhere other than util.h

Reviewed By: larryliu0820

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

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 61bc5d1.

kirklandsign pushed a commit to kirklandsign/executorch that referenced this pull request Jan 29, 2024
Summary:
Pull Request resolved: pytorch#1712

This new version:
- Can fail non-fatally
- Uses RAII to clean up the memory
- Uses separate files for lean and aten modes
- Has tests
- Lives somewhere other than util.h

Reviewed By: larryliu0820

Differential Revision: D52645309

fbshipit-source-id: b117269046efb137982b214b6877b13376b721db
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants