Content-Length: 392295 | pFad | http://github.com/pytorch/pytorch/pull/154080

E0 [libc++ readiness][caffe2] No reason to check for "ext/stdio_filebuf.h" by yuxuanchen1997 · Pull Request #154080 · pytorch/pytorch · GitHub
Skip to content

[libc++ readiness][caffe2] No reason to check for "ext/stdio_filebuf.h" #154080

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

Closed
wants to merge 1 commit into from

Conversation

yuxuanchen1997
Copy link
Contributor

@yuxuanchen1997 yuxuanchen1997 commented May 21, 2025

Summary: There should be no reason to check for existence of this GNU C++ header here in this file. It doesn't include it. Removing this condition to make it build under libc++.

Differential Revision: D75179136

Copy link

pytorch-bot bot commented May 21, 2025

This appears to be a diff that was exported from phabricator, but the PR author does not have sufficient permissions to run CI. @yuxuanchen1997, please do step 2 of internal wiki to get write access so you do not need to get CI approvals in the future. If you think this is a mistake, please contact the Pytorch Dev Infra team.

Copy link

pytorch-bot bot commented May 21, 2025

🔗 Helpful Links

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

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

✅ No Failures

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

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

Copy link

linux-foundation-easycla bot commented May 21, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: yuxuanchen1997 / name: Yuxuan Chen (dd75884)

@facebook-github-bot
Copy link
Contributor

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

@yuxuanchen1997
Copy link
Contributor Author

@pytorchbot label "topic: not user facing"

@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label May 21, 2025
yuxuanchen1997 added a commit to yuxuanchen1997/pytorch that referenced this pull request May 21, 2025
…h" (pytorch#154080)

Summary:

There should be no reason to check for existence of this GNU C++ header here in this file. It doesn't include it. Removing this condition makes adfinder linkable under libc++ platform.

Test Plan: CI, and build adfinder.

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

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

yuxuanchen1997 added a commit to yuxuanchen1997/pytorch that referenced this pull request May 21, 2025
…h" (pytorch#154080)

Summary:
Pull Request resolved: pytorch#154080

There should be no reason to check for existence of this GNU C++ header here in this file. It doesn't include it. Removing this condition makes adfinder linkable under libc++ platform.

Test Plan: CI, and build adfinder.

Differential Revision: D75179136
@yuxuanchen1997 yuxuanchen1997 force-pushed the export-D75179136 branch 2 times, most recently from d2e8a6b to 156f2ed Compare May 22, 2025 03:39
yuxuanchen1997 added a commit to yuxuanchen1997/pytorch that referenced this pull request May 22, 2025
…h" (pytorch#154080)

Summary:

There should be no reason to check for existence of this GNU C++ header here in this file. It doesn't include it. Removing this condition makes adfinder linkable under libc++ platform.

Test Plan: CI, and build adfinder.

Differential Revision: D75179136
…h" (pytorch#154080)

Summary:
Pull Request resolved: pytorch#154080

There should be no reason to check for existence of this GNU C++ header here in this file. It doesn't include it. Removing this condition makes adfinder linkable under libc++ platform.

Test Plan: CI, and build adfinder.

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

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

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label May 22, 2025
@facebook-github-bot
Copy link
Contributor

@pytorchbot merge

(Initiating merge automatically since Phabricator Diff has merged)

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/trunk Trigger trunk jobs on your pull request fb-exported Merged topic: not user facing topic category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: http://github.com/pytorch/pytorch/pull/154080

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy