Uh oh!
There was an error while loading. Please reload this page.
Exposing Float Masks - #154372
Conversation
This comment has been minimized.
This comment has been minimized.
folkertdev
commented
Apr 18, 2026
This seems useful, do ask if you need help here |
rustbot
commented
Apr 22, 2026
r? @scottmcm rustbot has assigned @scottmcm. Use Why was this reviewer chosen?The reviewer was selected based on:
|
rustbot
commented
Apr 22, 2026
|
Apersoma
commented
Apr 22, 2026
@folkertdev would you be interested in reviewing this? |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
rustbot
commented
Apr 22, 2026
Reminder, once the PR becomes ready for a review, use |
i'll squash it at the end |
Apersoma
commented
Apr 23, 2026
@rustbot ready |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
tgross35
commented
Apr 23, 2026
When you do squash, please make sure the commit message says what the change does similar to the PR message. Currently it's "Added the symbol" :) Unfortunately we have no squash+merge on this repo so the commits are what shows up in the history and blame. |
Apersoma
commented
Apr 24, 2026
@rustbot ready |
Apersoma
commented
Apr 24, 2026
@tgross35 |
tgross35
commented
Apr 25, 2026
These are straightforward and likely pretty unobjectionable, so waiting a few releases would probably be reasonable.
I always prefer a PR first so we can see what changes. It can be nominated, then FCP gets started after the meeting. |
Exposing Float Masks Tracking issue: rust-lang#154064 ACP: rust-lang/libs-team#753
Rollup of 7 pull requests Successful merges: - #155643 (Improve suggestion for $-prefixed fragment specifiers) - #154197 (Avoid redundant clone suggestions in borrowck diagnostics) - #154372 (Exposing Float Masks) - #155680 (Handle index projections in call destinations in DSE) - #155732 (bootstrap: Don't clone submodules unconditionally in dry-run) - #155737 (Account for `GetSyntheticValue` failures) - #155738 (Pass fields to `is_tuple_fields` instead of `SBValue` object)
Uh oh!
There was an error while loading. Please reload this page.
Rollup merge of #154372 - Apersoma:float_masks, r=tgross35 Exposing Float Masks Tracking issue: #154064 ACP: rust-lang/libs-team#753
View all comments
Tracking issue: #154064
ACP: rust-lang/libs-team#753