-
Notifications
You must be signed in to change notification settings - Fork 550
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.
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
Format packing semantics, dx12 handling #2037
Conversation
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.
Looks good, small nit.
(Discussed in gitter)
dim: $dim, | ||
packed: $total > 0x1000, |
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.
Quite a bit of magic going on here but looks ok for now :p
@@ -40,6 +40,11 @@ pub struct FormatDesc { | |||
pub bits: u16, | |||
/// Dimensions (width, height) of the texel blocks. | |||
pub dim: (u8, u8), | |||
/// The format representation depends on the endianness of the platform. | |||
/// | |||
/// * On little-endian systems, the actual oreder of components is reverse of what |
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.
nit: oreder/order
Thanks for the review!
Can't wait to make that portability pr, so let's proceed and follow up later with nit fixes.
Bors r=msiglreith
… On May 17, 2018, at 09:02, msiglreith ***@***.***> wrote:
@msiglreith approved this pull request.
Looks good, small nit.
(Discussed in gitter)
In src/hal/src/format.rs:
> dim: $dim,
+ packed: $total > 0x1000,
Quite a bit of magic going on here but looks ok for now :p
In src/hal/src/format.rs:
> @@ -40,6 +40,11 @@ pub struct FormatDesc {
pub bits: u16,
/// Dimensions (width, height) of the texel blocks.
pub dim: (u8, u8),
+ /// The format representation depends on the endianness of the platform.
+ ///
+ /// * On little-endian systems, the actual oreder of components is reverse of what
nit: oreder/order
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
2037: Format packing semantics, dx12 handling r=msiglreith a=kvark Fixes all of the image related copy/blit CTS tests (>11K passes!). There are 0 failures in this category now and ~5 crashes due to #1945. PR checklist: - [ ] `make` succeeds (on *nix) - [x] `make reftests` succeeds - [ ] tested examples with the following backends: Co-authored-by: Dzmitry Malyshau <dmalyshau@mozilla.com>
Fixes all of the image related copy/blit CTS tests (>11K passes!). There are 0 failures in this category now and ~5 crashes due to #1945.
PR checklist:
make
succeeds (on *nix)make reftests
succeeds