Skip to content

Fix compilation errors appearing due to missing <cstdint> include #81233

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kovdan01
Copy link
Contributor

@kovdan01 kovdan01 commented May 1, 2025

I've recently started to face compilation issues related to types like uint32_t not found. The issues remained even when rolling back to Apr 15 state of the repositories - at that point, no issues were present.

Compilation errors have probably started to occur after my Linux system LLVM installation was recently updated to 19.1.7.

This patch resolves the issue by adding missing <cstdint> includes. For SmallVector.h, such a change was already introduced in LLVM repository: llvm/llvm-project@7e44305

I've recently started to face compilation issues related to types like
`uint32_t` not found. The issues remained even when rolling back to Apr 15
state of the repositories - at that point, no issues were present.

Compilation errors are probably related to my system LLVM installation
being recently updated to 19.1.7.

This patch resolves the issue by adding missing `<cstdint>` includes.
@kovdan01 kovdan01 marked this pull request as ready for review May 1, 2025 10:55
@kovdan01 kovdan01 requested review from a team, hborla, slavapestov and xedin as code owners May 1, 2025 10:55
@kovdan01
Copy link
Contributor Author

kovdan01 commented May 1, 2025

Tagging @asl

@kovdan01 kovdan01 changed the title Fix compilation errors due to missing <cstdint> include Fix compilation errors appearing due to missing <cstdint> include May 1, 2025
@asl
Copy link
Contributor

asl commented May 1, 2025

@swift-ci please test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants