Skip to content

Conversation

@ZuseZ4
Copy link
Member

@ZuseZ4 ZuseZ4 commented Dec 4, 2025

@rustbot

This comment was marked as outdated.

@rustbot rustbot added A-compiler-builtins Area: compiler-builtins (https://github.com/rust-lang/compiler-builtins) S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Dec 4, 2025
@rustbot
Copy link
Collaborator

rustbot commented Dec 4, 2025

workingjubilee is currently at their maximum review capacity.
They may take a while to respond.

Copy link
Contributor

@juntyr juntyr left a comment

Choose a reason for hiding this comment

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

LGTM and matches the PTX ISA docs

View changes since this review

@workingjubilee
Copy link
Member

hm? shouldn't this PR be made against the stdarch repo, since it only affects that subtree? I can accept it, but

@ZuseZ4
Copy link
Member Author

ZuseZ4 commented Dec 9, 2025

First time contributing to it, sure. New PR: rust-lang/stdarch#1973

@ZuseZ4 ZuseZ4 closed this Dec 9, 2025
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Dec 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-compiler-builtins Area: compiler-builtins (https://github.com/rust-lang/compiler-builtins) T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants