Skip to content

Upgrade torchao build configs to C++20 (#4025)#4025

Merged
meta-codesync[bot] merged 1 commit intopytorch:mainfrom
r-barnes:export-D95103084
Mar 23, 2026
Merged

Upgrade torchao build configs to C++20 (#4025)#4025
meta-codesync[bot] merged 1 commit intopytorch:mainfrom
r-barnes:export-D95103084

Conversation

@r-barnes
Copy link
Copy Markdown
Contributor

@r-barnes r-barnes commented Mar 8, 2026

Summary:

Update CMAKE_CXX_STANDARD and compiler flags from c++17 to c++20
in torchao's setup.py, CMakeLists.txt, and Makefile files.

Reviewed By: andrewor14

Differential Revision: D95103084

@pytorch-bot
Copy link
Copy Markdown

pytorch-bot Bot commented Mar 8, 2026

🔗 Helpful Links

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

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

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

@meta-cla meta-cla 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 Mar 8, 2026
@andrewor14 andrewor14 added the topic: for developers Use this tag if this PR is mainly developer facing label Mar 9, 2026
@r-barnes r-barnes added the module: not user facing Use this tag if you don't want this PR to show up in release notes label Mar 12, 2026
@r-barnes
Copy link
Copy Markdown
Contributor Author

@pytorchbot merge

@pytorchmergebot
Copy link
Copy Markdown
Collaborator

Merge failed

Reason: 1 mandatory check(s) are pending/not yet run. The first few are:

  • Facebook CLA Check

Dig deeper by viewing the pending checks on hud

Details for Dev Infra team Raised by workflow job

Failing merge rule: superuser

@r-barnes
Copy link
Copy Markdown
Contributor Author

@pytorchbot merge -f "CLA satisfied"

@pytorchmergebot
Copy link
Copy Markdown
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use -f as last resort and instead consider -i/--ignore-current to continue the merge ignoring current failures. This will allow currently pending tests to finish and report signal before the merge.

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

@pytorchmergebot
Copy link
Copy Markdown
Collaborator

Merge failed

Reason: 1 mandatory check(s) are pending/not yet run. The first few are:

  • Facebook CLA Check

Dig deeper by viewing the pending checks on hud

Details for Dev Infra team Raised by workflow job

Failing merge rule: superuser

r-barnes added a commit to r-barnes/ao that referenced this pull request Mar 12, 2026
Summary:

Update CMAKE_CXX_STANDARD and compiler flags from c++17 to c++20
in torchao's setup.py, CMakeLists.txt, and Makefile files.

Reviewed By: andrewor14

Differential Revision: D95103084
@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented Mar 12, 2026

@r-barnes has exported this pull request. If you are a Meta employee, you can view the originating Diff in D95103084.

@meta-codesync meta-codesync Bot changed the title Upgrade torchao build configs to C++20 Upgrade torchao build configs to C++20 (#4025) Mar 14, 2026
r-barnes added a commit to r-barnes/ao that referenced this pull request Mar 14, 2026
Summary:

Update CMAKE_CXX_STANDARD and compiler flags from c++17 to c++20
in torchao's setup.py, CMakeLists.txt, and Makefile files.

Reviewed By: andrewor14

Differential Revision: D95103084
@r-barnes
Copy link
Copy Markdown
Contributor Author

If regression tests don't fail after rebase, we should be good.

r-barnes added a commit to r-barnes/ao that referenced this pull request Mar 14, 2026
Summary:

Update CMAKE_CXX_STANDARD and compiler flags from c++17 to c++20
in torchao's setup.py, CMakeLists.txt, and Makefile files.

Reviewed By: andrewor14

Differential Revision: D95103084
r-barnes added a commit to r-barnes/ao that referenced this pull request Mar 14, 2026
Summary:
Pull Request resolved: pytorch#4025

Update CMAKE_CXX_STANDARD and compiler flags from c++17 to c++20
in torchao's setup.py, CMakeLists.txt, and Makefile files.

Reviewed By: andrewor14

Differential Revision: D95103084
r-barnes added a commit to r-barnes/ao that referenced this pull request Mar 22, 2026
Summary:

Update CMAKE_CXX_STANDARD and compiler flags from c++17 to c++20
in torchao's setup.py, CMakeLists.txt, and Makefile files.

Reviewed By: andrewor14

Differential Revision: D95103084
r-barnes added a commit to r-barnes/ao that referenced this pull request Mar 22, 2026
Summary:
Pull Request resolved: pytorch#4025

Update CMAKE_CXX_STANDARD and compiler flags from c++17 to c++20
in torchao's setup.py, CMakeLists.txt, and Makefile files.

Reviewed By: andrewor14

Differential Revision: D95103084
Summary:

Update CMAKE_CXX_STANDARD and compiler flags from c++17 to c++20
in torchao's setup.py, CMakeLists.txt, and Makefile files.

Reviewed By: andrewor14

Differential Revision: D95103084
@meta-codesync meta-codesync Bot merged commit fbce917 into pytorch:main Mar 23, 2026
42 of 43 checks passed
pytorch-bot Bot pushed a commit to pytorch/pytorch that referenced this pull request Apr 14, 2026
Bump the torchao commit pin to fbce9178 which includes
pytorch/ao#4025 (Upgrade torchao build configs to C++20).
This is needed because #178150 enforces C++20 in ATen.h
header guards, and the old pin built CUDA extensions with
-std=c++17, causing compilation failures in the inductor CI.
r-barnes added a commit to r-barnes/pytorch that referenced this pull request Apr 14, 2026
Bump the torchao commit pin to fbce9178 which includes
pytorch/ao#4025 (Upgrade torchao build configs to C++20).
This is needed because pytorch#178150 enforces C++20 in ATen.h
header guards, and the old pin built CUDA extensions with
-std=c++17, causing compilation failures in the inductor CI.
pytorchmergebot pushed a commit to pytorch/pytorch that referenced this pull request Apr 14, 2026
## Summary
- Bump torchao pin from `985d970b` to `fbce9178` (pytorch/ao#4025)
- The new pin builds CUDA extensions with `-std=c++20` instead of `-std=c++17`
- Fixes the inductor build failure introduced by #178150 (C++20 header guards in ATen.h)

## Test plan
- Inductor CI build should pass (torchao CUDA extensions now compile with C++20)
Pull Request resolved: #180302
Approved by: https://github.com/Skylion007
BoyuanFeng pushed a commit to pytorch/pytorch that referenced this pull request Apr 15, 2026
## Summary
- Bump torchao pin from `985d970b` to `fbce9178` (pytorch/ao#4025)
- The new pin builds CUDA extensions with `-std=c++20` instead of `-std=c++17`
- Fixes the inductor build failure introduced by #178150 (C++20 header guards in ATen.h)

## Test plan
- Inductor CI build should pass (torchao CUDA extensions now compile with C++20)
Pull Request resolved: #180302
Approved by: https://github.com/Skylion007
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 meta-exported module: not user facing Use this tag if you don't want this PR to show up in release notes topic: for developers Use this tag if this PR is mainly developer facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants