Skip to content

Navigation Menu

Appearance settings

GitHub Copilot
Write better code with AI

GitHub Spark New
Build and deploy intelligent apps

GitHub Models New
Manage and compare prompts

GitHub Advanced Security
Find and fix vulnerabilities

Actions
Automate any workflow
Codespaces
Instant dev environments

Issues
Plan and track work

Code Review
Manage code changes

Discussions
Collaborate outside of code

Code Search
Find more, search less
Explore

Why GitHub

Documentation

GitHub Skills

Blog
Integrations

GitHub Marketplace

MCP Registry
View all features
By company size

Enterprises

Small and medium teams

Startups

Nonprofits
By use case

App Modernization

DevSecOps

DevOps

CI/CD

View all use cases
By industry

Healthcare

Financial services

Manufacturing

Government

View all industries
View all solutions
Topics

AI

DevOps

Security

Software Development

View all
Explore

Learning Pathways

Events & Webinars

Ebooks & Whitepapers

Customer Stories

Partners

Executive Insights
GitHub Sponsors
Fund open source developers
The ReadME Project
GitHub community articles
Repositories

Topics

Trending

Collections
Enterprise platform
AI-powered developer platform
Available add-ons

GitHub Advanced Security
Enterprise-grade security features

Copilot for business
Enterprise-grade AI features

Premium Support
Enterprise-grade 24/7 support
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

Appearance settings

You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.

Dismiss alert

llvm / llvm-project Public

Notifications You must be signed in to change notification settings
Fork 15.2k
Star 35.4k

Code
Issues 5k+
Pull requests 5k+
Actions
Security
Insights

Additional navigation options

Code
Issues
Pull requests
Actions
Security
Insights

[Clang] Add __builtin_counted_by_ref builtin#114495

Merged

bwendling merged 73 commits intollvm:mainllvm/llvm-project:mainfrom

bwendling:builtin_get_counted_bybwendling/llvm-project:builtin_get_counted_byCopy head branch name to clipboard

Nov 7, 2024

Conversation Commits73 (73)Checks Files changed

Merged

[Clang] Add __builtin_counted_by_ref builtin#114495
bwendling merged 73 commits intollvm:mainfrom
bwendling:builtin_get_counted_by

Commits

Commits on Nov 1, 2024

[Clang] Add __builtin_counted_by_ref builtin
bwendling
committed
If the 'counted_by' attribute doesn't exist, then return nullptr.
bwendling
committed
Remove debugging code.
bwendling
committed
Make returning a nullptr the default.
bwendling
committed
clang-format
bwendling
committed
Use a visitor to retrieve the MemberExpr. It's intentionally harsh and ignores pretty much everythings that may 'hide' the underlying MemberExpr.
bwendling
committed
Add testcases.
bwendling
committed
Combine the code to grab the MemberExpr, if it exists.
bwendling
committed
Move FindCountedByField into a central place.
bwendling
committed
Use CustomTypeChecking for the builtin. It allows us explicitly to set the return type based on the 'count' field's type.
bwendling
committed
[Clang][NFC] Move FindCountedByField into FieldDecl (#104235)
bwendling
committed
Fix a build failure.
bwendling
committed
Remove empty line.
bwendling
committed
Remove merge errors.
bwendling
committed
Use 'size_t *' for the default return type and improve the testcase.
bwendling
committed
Don't use a Visitor to retrieve the MemberExpr. Instead accept only non-pathological expressions.
bwendling
committed
Improve error messages. Make the side-effects error a warning.
bwendling
committed
Add _Generic checks.
bwendling
committed
Remove unrelated change
bwendling
committed
Use MemberExpr::Create instead of MemberExpr::CreateImplicit.
bwendling
committed
Regenerate
bwendling
committed
Add -W flag for warning.
bwendling
committed
Handle any anonymous structs surrounding the flexible array member and / or count field.
bwendling
committed
Add anonymous struct testcase.
bwendling
committed
Report a fatal error if we can't find the 'count' field.
bwendling
committed
Use 'cast' instead of 'dyn_cast' and default to 'void *' for the return type.

isanbard
authored and
bwendling
committed
A 'void *' can result in an error message about assigning to a 'void'. Go back to 'size_t *' as the default return type.
bwendling
committed
Add some documentation.
bwendling
committed
Rename to __builtin_counted_by_ref.
bwendling
committed
Rename method to reflect new builtin name.
bwendling
committed
Emit an error if the 'count' reference is assigned to a variable, used as a function arg, or in a return.
bwendling
committed
Ignore paren casts.
bwendling
committed
Don't allow __builtin_counted_by_ref in complex expressions.
bwendling
committed
Add test case for pointers.
bwendling
committed
Add SourceRange to the diagnostic.
bwendling
committed
Check that __builtin_counted_by_ref isn't used on the LHS by a complex equation.
bwendling
committed
Update clang/include/clang/Basic/DiagnosticSemaKinds.td

bwendling
and
AaronBallman
committed
Add a flag indicating that the 'MemberExpr' was once a call to '__builtin_counted_by_ref' so that we can issue proper diagnostics.
bwendling
committed
Remove invalid tests that take the address of the bounds counter.
bwendling
committed
Add testcase that discards side-effects.
bwendling
committed
Delay warnings for _Generic statements.
bwendling
committed
Marking some diagnostics as 'potentially evaluated' somehow prevented variables from being marked as 'used'. >_<
bwendling
committed
Regenerate testcase.
bwendling
committed
Fix testcase.
bwendling
committed
Fix some of the naming and remove dead code.
bwendling
committed
Correct include ordering.
bwendling
committed
Improve the argument checking so that it's checking specifically for 'counted_by' and not other counting attributes. Also make the documentation better.
bwendling
committed
Move setting that the counter is a bounds-safety counter to where __builtin_counted_by_ref is processed so that future sema checks are concerned only with that code path.
bwendling
committed
Retain the __builtin_counted_by_ref CallExpr for AST dumping reasons. We'll lower it during CodeGen.
bwendling
committed
Fix
bwendling
committed
Reformat
bwendling
committed
Add correct attribute checking and set the return type.
bwendling
committed
Generate LLVM IR for the __builtin_counted_by_ref.
bwendling
committed
Set InBounds on the GEP.
bwendling
committed
Regenerate testcase.
bwendling
committed
Remove warning that no longer shows up.
bwendling
committed
Regenerate testcase
bwendling
committed
Expand testing. This includes a hack to prevent multiple diagnostics for the same issue.
bwendling
committed
Add test rejecting using of 'counted_by' in C++.
bwendling
committed

Commits on Nov 4, 2024

Add -ast-print testcase.
bwendling
committed
Re-add inbounds to the GEP.
bwendling
committed
Add test for &ptr->array
bwendling
committed
Remote -O2 from codegen.
bwendling
committed
Remove setting the 'inBounds' flag.
bwendling
committed

Commits on Nov 5, 2024

Upgrade the side-effects warning into an error.
bwendling
committed
Restrict __builtin_counted_by_ref to 'ptr->array' to match GCC's implementation.
bwendling
committed
Regenerate.
bwendling
committed
Revert the modifications as they're not affected by this feature.
bwendling
committed

Commits on Nov 7, 2024

Add testcase.
bwendling
committed
Revert.
bwendling
committed
Revert unaffected tests.
bwendling
committed
Merge branch 'llvm:main' into builtin_get_counted_by
bwendling
authored
Fix Sphinx formatting.
bwendling
committed

Footer

© 2025 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Community
Docs
Contact

You can’t perform that action at this time.