Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add method to get FnAbi of function pointer #126722

Merged
merged 1 commit into from
Jun 22, 2024
Merged

Add method to get FnAbi of function pointer #126722

merged 1 commit into from
Jun 22, 2024

Conversation

adwinwhite
Copy link
Contributor

Provide a StableMIR API to query FnAbi of a function pointer.

Fixes rust-lang/project-stable-mir#63

@rustbot
Copy link
Collaborator

rustbot commented Jun 20, 2024

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @celinval (or someone else) some time within the next two weeks.

Please see the contribution instructions for more information. Namely, in order to ensure the minimum review times lag, PR authors and assigned reviewers should ensure that the review label (S-waiting-on-review and S-waiting-on-author) stays updated, invoking these commands when appropriate:

  • @rustbot author: the review is finished, PR author should check the comments and take action accordingly
  • @rustbot review: the author is ready for a review, this PR will be queued again in the reviewer's queue

@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 Jun 20, 2024
@rustbot
Copy link
Collaborator

rustbot commented Jun 20, 2024

This PR changes Stable MIR

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

celinval
celinval reviewed Jun 20, 2024
View reviewed changes
Copy link
Contributor

@celinval celinval left a comment

Choose a reason for hiding this comment

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

Looks good! Just one minor comment

compiler/stable_mir/src/ty.rs Outdated Show resolved Hide resolved
@celinval
Copy link
Contributor

Awesome! Thanks

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Jun 21, 2024

📌 Commit 225796a has been approved by celinval

It is now in the queue for this repository.

@bors bors 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 Jun 21, 2024
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jun 22, 2024
@matthiaskrgr
Rollup merge of rust-lang#126722 - adwinwhite:ptr_fn_abi, r=celinval
e843355 
Add method to get `FnAbi` of function pointer

Provide a StableMIR API to query `FnAbi` of a function pointer.

Fixes [rust-lang/project-stable-mir#63](rust-lang/project-stable-mir#63)
@matthiaskrgr matthiaskrgr mentioned this pull request Jun 22, 2024
Closed
bors added a commit to rust-lang-ci/rust that referenced this pull request Jun 22, 2024
@bors
Auto merge of rust-lang#126815 - matthiaskrgr:rollup-yulic2r, r=matth…
a9adb67 
…iaskrgr

Rollup of 6 pull requests

Successful merges:

 - rust-lang#116113 ( Generalize `{Rc,Arc}::make_mut()` to unsized types.)
 - rust-lang#126722 (Add method to get `FnAbi` of function pointer)
 - rust-lang#126787 (Add direct accessors for memory addresses in `Machine` (for Miri))
 - rust-lang#126798 ([fuchsia-test-runner] Remove usage of kw_only)
 - rust-lang#126809 (Remove stray `.` from error message)
 - rust-lang#126811 (Add a tidy rule to check that fluent messages and attrs don't end in `.`)

r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit to rust-lang-ci/rust that referenced this pull request Jun 22, 2024
@bors
Auto merge of rust-lang#126815 - matthiaskrgr:rollup-yulic2r, r=<try>
d8ec120 
Rollup of 6 pull requests

Successful merges:

 - rust-lang#116113 ( Generalize `{Rc,Arc}::make_mut()` to unsized types.)
 - rust-lang#126722 (Add method to get `FnAbi` of function pointer)
 - rust-lang#126787 (Add direct accessors for memory addresses in `Machine` (for Miri))
 - rust-lang#126798 ([fuchsia-test-runner] Remove usage of kw_only)
 - rust-lang#126809 (Remove stray `.` from error message)
 - rust-lang#126811 (Add a tidy rule to check that fluent messages and attrs don't end in `.`)

r? `@ghost`
`@rustbot` modify labels: rollup

try-job: test-various
@workingjubilee workingjubilee mentioned this pull request Jun 22, 2024
Merged
bors added a commit to rust-lang-ci/rust that referenced this pull request Jun 22, 2024
@bors
Auto merge of rust-lang#126817 - workingjubilee:rollup-0rg0k55, r=wor…
f0aceed 
…kingjubilee

Rollup of 7 pull requests

Successful merges:

 - rust-lang#126530 (Add `f16` inline ASM support for RISC-V)
 - rust-lang#126712 (Migrate `relocation-model`, `error-writing-dependencies` and `crate-name-priority` `run-make` tests to rmake)
 - rust-lang#126722 (Add method to get `FnAbi` of function pointer)
 - rust-lang#126787 (Add direct accessors for memory addresses in `Machine` (for Miri))
 - rust-lang#126798 ([fuchsia-test-runner] Remove usage of kw_only)
 - rust-lang#126809 (Remove stray `.` from error message)
 - rust-lang#126811 (Add a tidy rule to check that fluent messages and attrs don't end in `.`)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 1f9793f into rust-lang:master Jun 22, 2024
6 checks passed
@rustbot rustbot added this to the 1.81.0 milestone Jun 22, 2024
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Jun 22, 2024
@rust-timer
Unrolled build for rust-lang#126722
008605b 
Rollup merge of rust-lang#126722 - adwinwhite:ptr_fn_abi, r=celinval

Add method to get `FnAbi` of function pointer

Provide a StableMIR API to query `FnAbi` of a function pointer.

Fixes [rust-lang/project-stable-mir#63](rust-lang/project-stable-mir#63)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@celinval celinval celinval left review comments

Assignees

@celinval celinval

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
Milestone
1.81.0
Development

Successfully merging this pull request may close these issues.

Add a method to get the FnAbi of a function pointer
4 participants
@adwinwhite @rustbot @celinval @bors