-
Notifications
You must be signed in to change notification settings - Fork 13.5k
Add Param and Bound ty to SMIR #113930
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Add Param and Bound ty to SMIR #113930
Conversation
This PR changes Stable MIR cc @oli-obk, @celinval, @spastorino |
☔ The latest upstream changes (presumably #113943) made this pull request unmergeable. Please resolve the merge conflicts. |
☔ The latest upstream changes (presumably #114024) made this pull request unmergeable. Please resolve the merge conflicts. |
☔ The latest upstream changes (presumably #114076) made this pull request unmergeable. Please resolve the merge conflicts. |
@@ -18,6 +18,8 @@ type Span = Opaque; | |||
pub enum TyKind { | |||
RigidTy(RigidTy), | |||
Alias(AliasKind, AliasTy), | |||
Param(ParamTy), | |||
Bound(usize, BoundTy), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe make this a struct variant and give names to the fields.
@bors r+ rollup @spastorino said they'd follow up with doing such changes across the board instead of just in this instance |
Add Param and Bound ty to SMIR r? `@oli-obk`
…iaskrgr Rollup of 7 pull requests Successful merges: - rust-lang#101994 (rand: freebsd update, using getrandom.) - rust-lang#113930 (Add Param and Bound ty to SMIR) - rust-lang#113942 (Squelch a noisy rustc_expand unittest) - rust-lang#113996 (Define CMAKE_SYSTEM_NAME on a cross build targeting DragonFly.) - rust-lang#114070 (Add `sym::iter_mut` + `sym::as_mut_ptr` for Clippy) - rust-lang#114073 (Remove -Z diagnostic-width) - rust-lang#114090 (compiletest: remove ci-specific remap-path-prefix) r? `@ghost` `@rustbot` modify labels: rollup
r? @oli-obk