Skip to content
This repository was archived by the owner on Feb 21, 2026. It is now read-only.

[CIR] Remove explicit checks of function attributes in unrelated tests#2027

Merged
xlauko merged 1 commit intomainfrom
users/xlauko/simplify-function-attribute-checks
Nov 28, 2025
Merged

[CIR] Remove explicit checks of function attributes in unrelated tests#2027
xlauko merged 1 commit intomainfrom
users/xlauko/simplify-function-attribute-checks

Conversation

@xlauko
Copy link
Copy Markdown
Collaborator

@xlauko xlauko commented Nov 26, 2025

This simplifies the transition to explicit attributes by removing the need to update tests that don’t care about every individual attribute. Only the tests that are specifically intended to check those attributes still use explicit attribute checks.

@xlauko
Copy link
Copy Markdown
Collaborator Author

xlauko commented Nov 26, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@xlauko xlauko marked this pull request as ready for review November 26, 2025 21:05
@xlauko xlauko force-pushed the users/xlauko/simplify-function-attribute-checks branch from 9e47299 to b292185 Compare November 26, 2025 21:07
@xlauko xlauko force-pushed the users/xlauko/simplify-function-attribute-checks branch from b292185 to 48cffd8 Compare November 28, 2025 07:58
@xlauko xlauko requested a review from lanza as a code owner November 28, 2025 07:58
@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 28, 2025

✅ With the latest revision this PR passed the C/C++ code formatter.

@xlauko xlauko force-pushed the users/xlauko/simplify-function-attribute-checks branch from 48cffd8 to 9d0a784 Compare November 28, 2025 08:05
@xlauko xlauko force-pushed the users/xlauko/simplify-function-attribute-checks branch from 9d0a784 to 81c46ae Compare November 28, 2025 08:15
This will simplify transition to other explicit attributes from extra attrs.
@xlauko xlauko force-pushed the users/xlauko/simplify-function-attribute-checks branch from 81c46ae to 82422eb Compare November 28, 2025 09:00
@xlauko xlauko merged commit 560e522 into main Nov 28, 2025
6 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants