ast: Remove pointless deleted copy constructors
ast::Node has a deleted copy constructor, so the derived classes cannot be implicitly copied.
Change-Id: I57f74a4ab35dd00b3290b964cb6492f4fa4505a5
Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/66602
Commit-Queue: Ben Clayton <bclayton@google.com>
Kokoro: Ben Clayton <bclayton@google.com>
Kokoro: Kokoro <noreply+kokoro@google.com>
Reviewed-by: James Price <jrprice@google.com>
diff --git a/src/ast/array_accessor_expression.h b/src/ast/array_accessor_expression.h
index fb5e5bc..543ec1d 100644
--- a/src/ast/array_accessor_expression.h
+++ b/src/ast/array_accessor_expression.h
@@ -48,9 +48,6 @@
/// the index expression
Expression* const index;
-
- private:
- ArrayAccessorExpression(const ArrayAccessorExpression&) = delete;
};
} // namespace ast
diff --git a/src/ast/assignment_statement.h b/src/ast/assignment_statement.h
index bb8ea25..c6fdc5a 100644
--- a/src/ast/assignment_statement.h
+++ b/src/ast/assignment_statement.h
@@ -48,9 +48,6 @@
/// right side expression
Expression* const rhs;
-
- private:
- AssignmentStatement(const AssignmentStatement&) = delete;
};
} // namespace ast
diff --git a/src/ast/binary_expression.h b/src/ast/binary_expression.h
index 8c088f4..bca7aa4 100644
--- a/src/ast/binary_expression.h
+++ b/src/ast/binary_expression.h
@@ -118,9 +118,6 @@
Expression* const lhs;
/// the right side expression
Expression* const rhs;
-
- private:
- BinaryExpression(const BinaryExpression&) = delete;
};
inline bool BinaryExpression::IsArithmetic() const {
diff --git a/src/ast/bitcast_expression.h b/src/ast/bitcast_expression.h
index 6a0a60a..a599679 100644
--- a/src/ast/bitcast_expression.h
+++ b/src/ast/bitcast_expression.h
@@ -49,9 +49,6 @@
ast::Type* const type;
/// the expression
Expression* const expr;
-
- private:
- BitcastExpression(const BitcastExpression&) = delete;
};
} // namespace ast
diff --git a/src/ast/block_statement.h b/src/ast/block_statement.h
index f37db9e..f8f39b1 100644
--- a/src/ast/block_statement.h
+++ b/src/ast/block_statement.h
@@ -52,9 +52,6 @@
/// the statement list
StatementList const statements;
-
- private:
- BlockStatement(const BlockStatement&) = delete;
};
} // namespace ast
diff --git a/src/ast/break_statement.h b/src/ast/break_statement.h
index 2bc74f9..15aeebe 100644
--- a/src/ast/break_statement.h
+++ b/src/ast/break_statement.h
@@ -36,9 +36,6 @@
/// @param ctx the clone context
/// @return the newly cloned node
BreakStatement* Clone(CloneContext* ctx) const override;
-
- private:
- BreakStatement(const BreakStatement&) = delete;
};
} // namespace ast
diff --git a/src/ast/call_expression.h b/src/ast/call_expression.h
index 7cf625e..6693e2f 100644
--- a/src/ast/call_expression.h
+++ b/src/ast/call_expression.h
@@ -49,9 +49,6 @@
IdentifierExpression* const func;
/// The arguments
ExpressionList const args;
-
- private:
- CallExpression(const CallExpression&) = delete;
};
} // namespace ast
diff --git a/src/ast/call_statement.h b/src/ast/call_statement.h
index 2eb5aa3..130e7f5 100644
--- a/src/ast/call_statement.h
+++ b/src/ast/call_statement.h
@@ -41,9 +41,6 @@
/// The call expression
CallExpression* const expr;
-
- private:
- CallStatement(const CallStatement&) = delete;
};
} // namespace ast
diff --git a/src/ast/case_statement.h b/src/ast/case_statement.h
index eb74d59..ed96a11 100644
--- a/src/ast/case_statement.h
+++ b/src/ast/case_statement.h
@@ -56,9 +56,6 @@
/// The case body
BlockStatement* const body;
-
- private:
- CaseStatement(const CaseStatement&) = delete;
};
/// A list of case statements
diff --git a/src/ast/constructor_expression.h b/src/ast/constructor_expression.h
index 93a19fd..fc19e98 100644
--- a/src/ast/constructor_expression.h
+++ b/src/ast/constructor_expression.h
@@ -33,9 +33,6 @@
ConstructorExpression(ProgramID pid, const Source& src);
/// Move constructor
ConstructorExpression(ConstructorExpression&&);
-
- private:
- ConstructorExpression(const ConstructorExpression&) = delete;
};
} // namespace ast
diff --git a/src/ast/continue_statement.h b/src/ast/continue_statement.h
index eaf799d..73445a3 100644
--- a/src/ast/continue_statement.h
+++ b/src/ast/continue_statement.h
@@ -36,9 +36,6 @@
/// @param ctx the clone context
/// @return the newly cloned node
ContinueStatement* Clone(CloneContext* ctx) const override;
-
- private:
- ContinueStatement(const ContinueStatement&) = delete;
};
} // namespace ast
diff --git a/src/ast/discard_statement.h b/src/ast/discard_statement.h
index 2159323..1284b75 100644
--- a/src/ast/discard_statement.h
+++ b/src/ast/discard_statement.h
@@ -36,9 +36,6 @@
/// @param ctx the clone context
/// @return the newly cloned node
DiscardStatement* Clone(CloneContext* ctx) const override;
-
- private:
- DiscardStatement(const DiscardStatement&) = delete;
};
} // namespace ast
diff --git a/src/ast/else_statement.h b/src/ast/else_statement.h
index 5cdc344..6dcc2c9 100644
--- a/src/ast/else_statement.h
+++ b/src/ast/else_statement.h
@@ -50,9 +50,6 @@
/// The else body
BlockStatement* const body;
-
- private:
- ElseStatement(const ElseStatement&) = delete;
};
/// A list of else statements
diff --git a/src/ast/expression.h b/src/ast/expression.h
index a319f16..b5a51f2 100644
--- a/src/ast/expression.h
+++ b/src/ast/expression.h
@@ -36,9 +36,6 @@
Expression(ProgramID pid, const Source& src);
/// Move constructor
Expression(Expression&&);
-
- private:
- Expression(const Expression&) = delete;
};
/// A list of expressions
diff --git a/src/ast/fallthrough_statement.h b/src/ast/fallthrough_statement.h
index 026a907..282869f 100644
--- a/src/ast/fallthrough_statement.h
+++ b/src/ast/fallthrough_statement.h
@@ -36,9 +36,6 @@
/// @param ctx the clone context
/// @return the newly cloned node
FallthroughStatement* Clone(CloneContext* ctx) const override;
-
- private:
- FallthroughStatement(const FallthroughStatement&) = delete;
};
} // namespace ast
diff --git a/src/ast/for_loop_statement.h b/src/ast/for_loop_statement.h
index 2dcd167..72b5026 100644
--- a/src/ast/for_loop_statement.h
+++ b/src/ast/for_loop_statement.h
@@ -59,9 +59,6 @@
/// The loop body block
BlockStatement* const body;
-
- private:
- ForLoopStatement(const ForLoopStatement&) = delete;
};
} // namespace ast
diff --git a/src/ast/function.h b/src/ast/function.h
index 2e60d55..a76c6e1 100644
--- a/src/ast/function.h
+++ b/src/ast/function.h
@@ -86,9 +86,6 @@
/// The decorations attached to the function return type.
DecorationList const return_type_decorations;
-
- private:
- Function(const Function&) = delete;
};
/// A list of functions
diff --git a/src/ast/identifier_expression.h b/src/ast/identifier_expression.h
index b60a5ad..7977b29 100644
--- a/src/ast/identifier_expression.h
+++ b/src/ast/identifier_expression.h
@@ -40,9 +40,6 @@
/// The symbol for the identifier
Symbol const symbol;
-
- private:
- IdentifierExpression(const IdentifierExpression&) = delete;
};
} // namespace ast
diff --git a/src/ast/if_statement.h b/src/ast/if_statement.h
index f955107..cd37ba0 100644
--- a/src/ast/if_statement.h
+++ b/src/ast/if_statement.h
@@ -54,9 +54,6 @@
/// The else statements
ElseStatementList const else_statements;
-
- private:
- IfStatement(const IfStatement&) = delete;
};
} // namespace ast
diff --git a/src/ast/loop_statement.h b/src/ast/loop_statement.h
index 062ad39..5f0ba50 100644
--- a/src/ast/loop_statement.h
+++ b/src/ast/loop_statement.h
@@ -47,9 +47,6 @@
/// The continuing statements
BlockStatement* const continuing;
-
- private:
- LoopStatement(const LoopStatement&) = delete;
};
} // namespace ast
diff --git a/src/ast/member_accessor_expression.h b/src/ast/member_accessor_expression.h
index a5c0bed..9716e2e 100644
--- a/src/ast/member_accessor_expression.h
+++ b/src/ast/member_accessor_expression.h
@@ -48,9 +48,6 @@
/// The member expression
IdentifierExpression* const member;
-
- private:
- MemberAccessorExpression(const MemberAccessorExpression&) = delete;
};
} // namespace ast
diff --git a/src/ast/return_statement.h b/src/ast/return_statement.h
index c6dd4a6..79e1b35 100644
--- a/src/ast/return_statement.h
+++ b/src/ast/return_statement.h
@@ -46,9 +46,6 @@
/// The value returned. May be null.
Expression* const value;
-
- private:
- ReturnStatement(const ReturnStatement&) = delete;
};
} // namespace ast
diff --git a/src/ast/scalar_constructor_expression.h b/src/ast/scalar_constructor_expression.h
index ad1d1ca..47412ac 100644
--- a/src/ast/scalar_constructor_expression.h
+++ b/src/ast/scalar_constructor_expression.h
@@ -44,9 +44,6 @@
/// The literal value
Literal* const literal;
-
- private:
- ScalarConstructorExpression(const ScalarConstructorExpression&) = delete;
};
} // namespace ast
diff --git a/src/ast/statement.h b/src/ast/statement.h
index 24e3cd2..bc31e37 100644
--- a/src/ast/statement.h
+++ b/src/ast/statement.h
@@ -37,9 +37,6 @@
Statement(ProgramID pid, const Source& src);
/// Move constructor
Statement(Statement&&);
-
- private:
- Statement(const Statement&) = delete;
};
/// A list of statements
diff --git a/src/ast/struct.h b/src/ast/struct.h
index beecd27..6235b2e 100644
--- a/src/ast/struct.h
+++ b/src/ast/struct.h
@@ -58,9 +58,6 @@
/// The struct decorations
DecorationList const decorations;
-
- private:
- Struct(const Struct&) = delete;
};
} // namespace ast
diff --git a/src/ast/struct_member.h b/src/ast/struct_member.h
index c3865b5..ac3725b 100644
--- a/src/ast/struct_member.h
+++ b/src/ast/struct_member.h
@@ -59,9 +59,6 @@
/// The decorations
DecorationList const decorations;
-
- private:
- StructMember(const StructMember&) = delete;
};
/// A list of struct members
diff --git a/src/ast/switch_statement.h b/src/ast/switch_statement.h
index beec419..8f9a8e7 100644
--- a/src/ast/switch_statement.h
+++ b/src/ast/switch_statement.h
@@ -51,8 +51,6 @@
/// The Switch body
CaseStatementList const body;
-
- private:
SwitchStatement(const SwitchStatement&) = delete;
};
diff --git a/src/ast/type_constructor_expression.h b/src/ast/type_constructor_expression.h
index f8590d4..b61d8dd 100644
--- a/src/ast/type_constructor_expression.h
+++ b/src/ast/type_constructor_expression.h
@@ -53,9 +53,6 @@
/// The values
ExpressionList const values;
-
- private:
- TypeConstructorExpression(const TypeConstructorExpression&) = delete;
};
} // namespace ast
diff --git a/src/ast/type_decl.h b/src/ast/type_decl.h
index df6b94d..2ad75c3 100644
--- a/src/ast/type_decl.h
+++ b/src/ast/type_decl.h
@@ -37,10 +37,6 @@
/// The name of the type declaration
Symbol const name;
-
- private:
- TypeDecl(const TypeDecl&) = delete;
- TypeDecl& operator=(const TypeDecl&) = delete;
};
} // namespace ast
diff --git a/src/ast/unary_op_expression.h b/src/ast/unary_op_expression.h
index 03345f5..baecf4a 100644
--- a/src/ast/unary_op_expression.h
+++ b/src/ast/unary_op_expression.h
@@ -48,9 +48,6 @@
/// The expression
Expression* const expr;
-
- private:
- UnaryOpExpression(const UnaryOpExpression&) = delete;
};
} // namespace ast
diff --git a/src/ast/variable.h b/src/ast/variable.h
index b02db6e..66d4811 100644
--- a/src/ast/variable.h
+++ b/src/ast/variable.h
@@ -152,9 +152,6 @@
/// The declared access control
Access const declared_access;
-
- private:
- Variable(const Variable&) = delete;
};
/// A list of variables
diff --git a/src/ast/variable_decl_statement.h b/src/ast/variable_decl_statement.h
index ef67ded..3335179 100644
--- a/src/ast/variable_decl_statement.h
+++ b/src/ast/variable_decl_statement.h
@@ -44,9 +44,6 @@
/// The variable
Variable* const variable;
-
- private:
- VariableDeclStatement(const VariableDeclStatement&) = delete;
};
} // namespace ast
