Skip to content

Third-party: Disable -Werror for flatcc build#19049

Merged
rascani merged 2 commits into
pytorch:mainfrom
rascani:fix-flatcc-werror-clang21
May 14, 2026
Merged

Third-party: Disable -Werror for flatcc build#19049
rascani merged 2 commits into
pytorch:mainfrom
rascani:fix-flatcc-werror-clang21

Conversation

@rascani
Copy link
Copy Markdown
Contributor

@rascani rascani commented Apr 22, 2026

Summary

Apple Clang 21 (Xcode 26) elevates -Wimplicit-int-conversion-on-negation and -Wunterminated-string-initialization to errors, which breaks the vendored flatcc build at pprintint.h and grisu3_print.h. The warnings are benign and the fix belongs upstream in dvidelabs/flatcc; disable -Werror locally via FLATCC_ALLOW_WERROR=OFF for both the flatcc_ep external project (host compiler) and the in-tree flatccrt runtime lib.

Test plan

./install_executorch.sh

Authored with Claude.

Apple Clang 21 (Xcode 26) elevates -Wimplicit-int-conversion-on-negation
and -Wunterminated-string-initialization to errors, which breaks the
vendored flatcc build at pprintint.h and grisu3_print.h. The warnings
are benign and the fix belongs upstream in dvidelabs/flatcc; disable
-Werror locally via FLATCC_ALLOW_WERROR=OFF for both the flatcc_ep
external project (host compiler) and the in-tree flatccrt runtime lib.

Authored with Claude.
@pytorch-bot
Copy link
Copy Markdown

pytorch-bot Bot commented Apr 22, 2026

🔗 Helpful Links

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

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

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

❌ 1 New Failure, 1 Pending

As of commit cf902a0 with merge base 5b89d23 (image):

NEW FAILURE - The following job has failed:

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 Apr 22, 2026
@github-actions
Copy link
Copy Markdown

This PR needs a release notes: label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

@rascani rascani requested a review from JacobSzwejbka April 22, 2026 21:52
@psiddh psiddh self-requested a review May 4, 2026 21:47
@rascani rascani merged commit 58b4f26 into pytorch:main May 14, 2026
173 of 174 checks passed
@rascani rascani deleted the fix-flatcc-werror-clang21 branch May 14, 2026 20:34
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants