Skip to content

Expose regular argument attributes in rustc_public - #163242

Merged
rust-bors[bot] merged 4 commits into
rust-lang:mainfrom
nihalpasham:rustc-public-arg-attributes
Oct 10, 2026
Merged

rust-bors[bot] merged 4 commits into
rust-lang:mainfrom
nihalpasham:rustc-public-arg-attributes

Conversation

@nihalpasham

@nihalpasham nihalpasham commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Add ArgAttributes::regular() and ArgAttributeFlags so consumers can read the argument flags already computed by rustc. The conversion preserves all ten flags, including the overlapping capture flags.

Clarifies what pointee_size guarantees at function entry and return, and how no_free affects rustc's emission of LLVM dereferenceability attributes.

Fixes rust-lang/rustc_public#132.

r? @makai410

@rustbot

rustbot commented Sep 24, 2026

Copy link
Copy Markdown
Collaborator

This PR changes rustc_public

cc @oli-obk, @celinval, @ouz-a, @makai410

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Sep 24, 2026
@rustbot

rustbot commented Sep 24, 2026

Copy link
Copy Markdown
Collaborator

Thanks for the pull request, and welcome! The Rust Project has assigned @makai410 (or someone else) to review your changes, you should hear from them (or someone else) within the next two weeks.

Please see the contribution instructions and our LLM policy for more information.

Comment thread compiler/rustc_public/src/abi.rs
Comment thread tests/ui-fulldeps/rustc_public/check_abi.rs Outdated
@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

Comment thread tests/ui-fulldeps/rustc_public/check_abi.rs
Comment thread tests/ui-fulldeps/rustc_public/check_abi.rs
@rust-bors

This comment has been minimized.

@nihalpasham
nihalpasham force-pushed the rustc-public-arg-attributes branch from ced5097 to 8a0c5b6 Compare October 10, 2026 06:07
@rustbot

rustbot commented Oct 10, 2026

Copy link
Copy Markdown
Collaborator

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@makai410 makai410 left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is looking good. Thanks!

@bors r+ rollup

View changes since this review

@rust-bors

rust-bors Bot commented Oct 10, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 8a0c5b6 has been approved by makai410

It is now in the queue for this repository.

@rust-bors rust-bors Bot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 10, 2026
rust-bors Bot pushed a commit that referenced this pull request Oct 10, 2026
Rollup of 4 pull requests

Successful merges:

 - #163434 (Add documentation for the `crate_name` attribute)
 - #163917 (Re-enable extension of bool return values on x86_64)
 - #163242 (Expose regular argument attributes in rustc_public)
 - #164071 (cg_llvm: Avoid all remaining uses of `as_c_char_ptr`)
@rust-bors
rust-bors Bot merged commit c087778 into rust-lang:main Oct 10, 2026
14 checks passed
@rustbot rustbot added this to the 1.101.0 milestone Oct 10, 2026
rust-bors Bot pushed a commit that referenced this pull request Oct 10, 2026
Rollup merge of #163242 - nihalpasham:rustc-public-arg-attributes, r=makai410

Expose regular argument attributes in rustc_public

Add `ArgAttributes::regular()` and `ArgAttributeFlags` so consumers can read the argument flags already computed by rustc. The conversion preserves all ten flags, including the overlapping capture flags.

Clarifies what `pointee_size` guarantees at function entry and return, and how `no_free` affects rustc's emission of LLVM dereferenceability attributes.

Fixes rust-lang/rustc_public#132.

r? @makai410
nihalpasham added a commit to NVIDIA/cuda-rust that referenced this pull request Oct 11, 2026
Update CUDAoxide from `nightly-2026-08-28` to `nightly-2026-10-11`. This
brings in rust-lang/rust#163242, needed for the separate kernel
parameter-attribute work in #1483 (related to #1088 and #1098).

## Changes

- Adapt to rustc API changes, generic floating-point intrinsics, and
integer min/max.
- Follow the new `core::intrinsics::reflection::type_id` path in the
kernel-name helper.
- Update toolchain/Nix pins, diagnostic snapshots, and intrinsic
evidence. Intrinsic declarations are unchanged.

## Validation

- CI: 55 passed, 7 expected skips; all 234 examples compile.
- Compiler tests, formatting, clippy, and IR comparison pass. Targeted
checks pass on both RTX 5090s with no memcheck errors.
- GPU sweep: 232/233 passed. The existing FP8 example failure is
addressed separately in #1487; hardware-limited cases retain
compile-only coverage.

The existing scaffold issue—mixing registry and Git runtime types—also
remains a separate follow-up.

## Checklist

- [x] All commits signed off
- [x] SPDX headers on new source files, where applicable

---------

Signed-off-by: nihalp <nihalp@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PassMode carries ArgAttributes only as Opaque; can we expose a typed mirror?

4 participants