Fixup whitespace/comments issues.
This CL fixes a few tabs to be spaces which makes the whitespace/comments lint check pass. The check is set to enabled. Change-Id: I2a6c590aa3bfee888239adc89d1ce4c8c99f3e78 Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/86083 Reviewed-by: Corentin Wallez <cwallez@chromium.org> Commit-Queue: Dan Sinclair <dsinclair@chromium.org> Auto-Submit: Dan Sinclair <dsinclair@chromium.org>
This commit is contained in:
parent
81e497c45f
commit
db9db31cab
|
@ -77,7 +77,7 @@ LINT_FILTERS = [
|
|||
"-legal/copyright", "-readability/braces", "-readability/casting",
|
||||
"-readability/inheritance", "-readability/namespace", "-runtime/explicit",
|
||||
"-runtime/indentation_namespace", "-runtime/int", "-runtime/printf",
|
||||
"-whitespace/comments", "-whitespace/empty_if_body"
|
||||
"-whitespace/empty_if_body"
|
||||
]
|
||||
|
||||
|
||||
|
|
|
@ -30,7 +30,7 @@ typedef __IOSurface* IOSurfaceRef;
|
|||
|
||||
#ifdef __OBJC__
|
||||
# import <Metal/Metal.h>
|
||||
#endif //__OBJC__
|
||||
#endif // __OBJC__
|
||||
|
||||
namespace dawn::native::metal {
|
||||
|
||||
|
|
|
@ -1690,9 +1690,9 @@ namespace {
|
|||
|
||||
// TODO (yunchao.he@intel.com):
|
||||
//
|
||||
// * Add tests for multiple encoders upon the same resource simultaneously. This situation fits
|
||||
// some cases like VR, multi-threading, etc.
|
||||
// * Add tests for multiple encoders upon the same resource simultaneously. This situation fits
|
||||
// some cases like VR, multi-threading, etc.
|
||||
//
|
||||
// * Add tests for bundle
|
||||
// * Add tests for bundle
|
||||
|
||||
} // anonymous namespace
|
||||
|
|
|
@ -134,11 +134,11 @@ namespace {
|
|||
}
|
||||
|
||||
// TODO (yunchao.he@intel.com):
|
||||
// * Add tests for compute, in which texture subresource is traced per dispatch.
|
||||
// * Add tests for compute, in which texture subresource is traced per dispatch.
|
||||
//
|
||||
// * Add tests for multiple encoders upon the same resource simultaneously. This situation fits
|
||||
// some cases like VR, multi-threading, etc.
|
||||
// * Add tests for multiple encoders upon the same resource simultaneously. This situation fits
|
||||
// some cases like VR, multi-threading, etc.
|
||||
//
|
||||
// * Add tests for conflicts between usages in two render bundles used in the same pass.
|
||||
// * Add tests for conflicts between usages in two render bundles used in the same pass.
|
||||
|
||||
} // anonymous namespace
|
||||
|
|
|
@ -181,16 +181,16 @@ class QueryInternalShaderTests : public DawnTest {
|
|||
}
|
||||
};
|
||||
|
||||
// Test the accuracy of timestamp compute shader which uses unsigned 32-bit integers to simulate
|
||||
// unsigned 64-bit integers (timestamps) multiplied by float (period).
|
||||
// The arguments pass to timestamp internal pipeline:
|
||||
// - The timestamps buffer contains the original timestamps resolved from query set (created
|
||||
// Test the accuracy of timestamp compute shader which uses unsigned 32-bit integers to simulate
|
||||
// unsigned 64-bit integers (timestamps) multiplied by float (period).
|
||||
// The arguments pass to timestamp internal pipeline:
|
||||
// - The timestamps buffer contains the original timestamps resolved from query set (created
|
||||
// manually here), and will be used to store the results processed by the compute shader.
|
||||
// Expect 0 for unavailable timestamps and nanoseconds for available timestamps in an expected
|
||||
// error tolerance ratio.
|
||||
// - The availability buffer passes the data of which slot in timestamps buffer is an initialized
|
||||
// - The availability buffer passes the data of which slot in timestamps buffer is an initialized
|
||||
// timestamp.
|
||||
// - The params buffer passes the timestamp count, the offset in timestamps buffer and the
|
||||
// - The params buffer passes the timestamp count, the offset in timestamps buffer and the
|
||||
// timestamp period (here use GPU frequency (HZ) on Intel D3D12 to calculate the period in
|
||||
// ns for testing).
|
||||
TEST_P(QueryInternalShaderTests, TimestampComputeShader) {
|
||||
|
|
Loading…
Reference in New Issue