Remove redundant constructors
Bug: chromium:42240266
Change-Id: Ie139029da0e864e595cf996cea8a652709891035
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/211950
Reviewed-by: Corentin Wallez <cwallez@google.com>
Commit-Queue: Jiawei Shao <jiawei.shao@intel.com>
diff --git a/src/dawn/native/PassResourceUsage.cpp b/src/dawn/native/PassResourceUsage.cpp
index 7c1cfa9..977d554 100644
--- a/src/dawn/native/PassResourceUsage.cpp
+++ b/src/dawn/native/PassResourceUsage.cpp
@@ -29,10 +29,6 @@
namespace dawn::native {
-ComputePassResourceUsage::ComputePassResourceUsage() = default;
-
-ComputePassResourceUsage::ComputePassResourceUsage(ComputePassResourceUsage&&) = default;
-
bool operator==(const TextureSyncInfo& a, const TextureSyncInfo& b) {
return (a.usage == b.usage) && (a.shaderStages == b.shaderStages);
}
diff --git a/src/dawn/native/PassResourceUsage.h b/src/dawn/native/PassResourceUsage.h
index 2333a20..f85642b 100644
--- a/src/dawn/native/PassResourceUsage.h
+++ b/src/dawn/native/PassResourceUsage.h
@@ -83,12 +83,6 @@
// textures used, because some unused BindGroups may not be used at all in synchronization
// scope but their resources still need to be validated on Queue::Submit.
struct ComputePassResourceUsage {
- // Somehow without this defaulted constructor, MSVC or its STDlib have an issue where they
- // use the copy constructor (that's deleted) when doing operations on a
- // vector<ComputePassResourceUsage>
- ComputePassResourceUsage(ComputePassResourceUsage&&);
- ComputePassResourceUsage();
-
std::vector<SyncScopeResourceUsage> dispatchUsages;
// All the resources referenced by this compute pass for validation in Queue::Submit.