Condense namespaces in tint/sem.

This PR condenses the namespaces in the tint/sem folder.

Change-Id: Iff802ca78d4cb7fd21134a640dfe5d04e85148b4
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/86030
Commit-Queue: Dan Sinclair <dsinclair@chromium.org>
Auto-Submit: Dan Sinclair <dsinclair@chromium.org>
Reviewed-by: Ben Clayton <bclayton@google.com>
Commit-Queue: Ben Clayton <bclayton@google.com>

src/tint/sem/array.cc

@@ -21,8 +21,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::Array);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 Array::Array(const Type* element,
              uint32_t count,
@@ -81,5 +80,4 @@ uint32_t Array::Size() const {
   return size_;
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/array.h

@@ -22,14 +22,11 @@
 #include "src/tint/sem/type.h"
 
 // Forward declarations
-namespace tint {
+namespace tint::ast {
-namespace ast {
 class Array;
-}  // namespace ast
+}  // namespace tint::ast
-}  // namespace tint
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// Array holds the semantic information for Array nodes.
 class Array final : public Castable<Array, Type> {
@@ -111,7 +108,6 @@ class Array final : public Castable<Array, Type> {
   const bool constructible_;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_ARRAY_H_

src/tint/sem/atomic_type.cc

@@ -20,8 +20,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::Atomic);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 Atomic::Atomic(const sem::Type* subtype) : subtype_(subtype) {
   TINT_ASSERT(AST, !subtype->Is<Reference>());
@@ -60,5 +59,4 @@ Atomic::Atomic(Atomic&&) = default;
 
 Atomic::~Atomic() = default;
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/atomic_type.h

@@ -19,8 +19,7 @@
 
 #include "src/tint/sem/type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// A atomic type.
 class Atomic final : public Castable<Atomic, Type> {
@@ -62,7 +61,6 @@ class Atomic final : public Castable<Atomic, Type> {
   sem::Type const* const subtype_;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_ATOMIC_TYPE_H_

src/tint/sem/atomic_type_test.cc

@@ -16,8 +16,7 @@
 
 #include "src/tint/sem/test_helper.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 using AtomicTest = TestHelper;
@@ -54,5 +53,4 @@ TEST_F(AtomicTest, FriendlyName) {
 }
 
 }  // namespace
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/behavior.cc

@@ -14,8 +14,7 @@
 
 #include "src/tint/sem/behavior.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 std::ostream& operator<<(std::ostream& out, Behavior behavior) {
   switch (behavior) {
@@ -35,5 +34,4 @@ std::ostream& operator<<(std::ostream& out, Behavior behavior) {
   return out << "<unknown>";
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/behavior.h

@@ -17,8 +17,7 @@
 
 #include "src/tint/utils/enum_set.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// Behavior enumerates the possible behaviors of an expression or statement.
 /// @see https://www.w3.org/TR/WGSL/#behaviors
@@ -40,7 +39,6 @@ using Behaviors = utils::EnumSet<Behavior>;
 /// @returns out so calls can be chained
 std::ostream& operator<<(std::ostream& out, Behavior behavior);
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_BEHAVIOR_H_

src/tint/sem/binding_point.h

@@ -21,8 +21,7 @@
 
 #include "src/tint/utils/hash.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// BindingPoint holds a group and binding index.
 struct BindingPoint {
@@ -46,8 +45,7 @@ struct BindingPoint {
   }
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 namespace std {
 

src/tint/sem/block_statement.cc

@@ -22,8 +22,7 @@ TINT_INSTANTIATE_TYPEINFO(tint::sem::BlockStatement);
 TINT_INSTANTIATE_TYPEINFO(tint::sem::FunctionBlockStatement);
 TINT_INSTANTIATE_TYPEINFO(tint::sem::LoopBlockStatement);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 BlockStatement::BlockStatement(const ast::BlockStatement* declaration,
                                const CompoundStatement* parent,
@@ -63,5 +62,4 @@ void LoopBlockStatement::SetFirstContinue(
   num_decls_at_first_continue_ = num_decls;
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/block_statement.h

@@ -21,17 +21,14 @@
 #include "src/tint/sem/statement.h"
 
 // Forward declarations
-namespace tint {
+namespace tint::ast {
-namespace ast {
 class BlockStatement;
 class ContinueStatement;
 class Function;
 class Variable;
-}  // namespace ast
+}  // namespace tint::ast
-}  // namespace tint
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// Holds semantic information about a block, such as parent block and variables
 /// declared in the block.
@@ -117,7 +114,6 @@ class LoopBlockStatement final
   size_t num_decls_at_first_continue_ = 0;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_BLOCK_STATEMENT_H_

src/tint/sem/bool_type.cc

@@ -18,8 +18,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::Bool);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 Bool::Bool() = default;
 
@@ -51,5 +50,4 @@ uint32_t Bool::Align() const {
   return 4;
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/bool_type.h

@@ -25,8 +25,7 @@
 #undef Bool
 #endif
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// A boolean type
 class Bool final : public Castable<Bool, Type> {
@@ -64,7 +63,6 @@ class Bool final : public Castable<Bool, Type> {
   uint32_t Align() const override;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_BOOL_TYPE_H_

src/tint/sem/bool_type_test.cc

@@ -15,8 +15,7 @@
 #include "src/tint/sem/test_helper.h"
 #include "src/tint/sem/texture_type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 using BoolTest = TestHelper;
@@ -46,5 +45,4 @@ TEST_F(BoolTest, FriendlyName) {
 }
 
 }  // namespace
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/builtin.cc

@@ -23,8 +23,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::Builtin);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 const char* Builtin::str() const {
   return sem::str(type_);
@@ -161,7 +160,6 @@ bool Builtin::HasSideEffects() const {
   return false;
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 //! @endcond

src/tint/sem/builtin.h

@@ -23,8 +23,7 @@
 #include "src/tint/sem/pipeline_stage_set.h"
 #include "src/tint/utils/hash.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// Determines if the given `i` is a coarse derivative
 /// @param i the builtin type
@@ -147,8 +146,7 @@ static constexpr double kRadToDeg = 57.295779513082322865;
 /// Constant value used by the radians() builtin
 static constexpr double kDegToRad = 0.017453292519943295474;
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 namespace std {
 

src/tint/sem/builtin_test.cc

@@ -16,8 +16,7 @@
 
 #include "gtest/gtest.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 struct BuiltinData {
@@ -124,5 +123,4 @@ TEST_F(BuiltinTypeTest, ParseNoMatch) {
 }
 
 }  // namespace
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/builtin_type.cc

@@ -26,8 +26,7 @@
 
 #include <sstream>
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 BuiltinType ParseBuiltinType(const std::string& name) {
   if (name == "abs") {
@@ -571,5 +570,4 @@ std::ostream& operator<<(std::ostream& out, BuiltinType i) {
   return out;
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/builtin_type.cc.tmpl

@@ -12,8 +12,7 @@ See:
 
 #include <sstream>
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 BuiltinType ParseBuiltinType(const std::string& name) {
 {{- range .Sem.Functions  }}
@@ -41,5 +40,4 @@ std::ostream& operator<<(std::ostream& out, BuiltinType i) {
   return out;
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/builtin_type.h

@@ -28,8 +28,7 @@
 #include <sstream>
 #include <string>
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// Enumerator of all builtin functions
 enum class BuiltinType {
@@ -155,7 +154,6 @@ const char* str(BuiltinType i);
 /// matches the name in the WGSL spec.
 std::ostream& operator<<(std::ostream& out, BuiltinType i);
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_BUILTIN_TYPE_H_

src/tint/sem/builtin_type.h.tmpl

@@ -14,8 +14,7 @@ See:
 #include <sstream>
 #include <string>
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// Enumerator of all builtin functions
 enum class BuiltinType {
@@ -39,7 +38,6 @@ const char* str(BuiltinType i);
 /// matches the name in the WGSL spec.
 std::ostream& operator<<(std::ostream& out, BuiltinType i);
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_BUILTIN_TYPE_H_

src/tint/sem/call.cc

@@ -19,8 +19,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::Call);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 Call::Call(const ast::CallExpression* declaration,
            const CallTarget* target,
@@ -38,5 +37,4 @@ Call::Call(const ast::CallExpression* declaration,
 
 Call::~Call() = default;
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/call.h

@@ -20,8 +20,7 @@
 #include "src/tint/sem/builtin.h"
 #include "src/tint/sem/expression.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// Call is the base class for semantic nodes that hold semantic information for
 /// ast::CallExpression nodes.
@@ -62,7 +61,6 @@ class Call final : public Castable<Call, Expression> {
   std::vector<const sem::Expression*> arguments_;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_CALL_H_

src/tint/sem/call_target.cc

@@ -19,8 +19,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::CallTarget);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 CallTarget::CallTarget(const sem::Type* return_type,
                        const ParameterList& parameters)
@@ -61,8 +60,7 @@ bool CallTargetSignature::operator==(const CallTargetSignature& other) const {
   return true;
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 namespace std {
 

src/tint/sem/call_target.h

@@ -22,10 +22,12 @@
 #include "src/tint/sem/variable.h"
 #include "src/tint/utils/hash.h"
 
-namespace tint {
-namespace sem {
 // Forward declarations
+namespace tint::sem {
 class Type;
+}
+
+namespace tint::sem {
 
 /// CallTargetSignature holds the return type and parameters for a call target
 struct CallTargetSignature {
@@ -84,8 +86,7 @@ class CallTarget : public Castable<CallTarget, Node> {
   CallTargetSignature signature_;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 namespace std {
 

src/tint/sem/constant.cc

@@ -21,8 +21,7 @@
 #include "src/tint/program_builder.h"
 #include "src/tint/sem/type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 namespace {
 
@@ -80,5 +79,4 @@ bool Constant::AnyZero() const {
   return false;
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/constant.h

@@ -20,8 +20,7 @@
 #include "src/tint/program_builder.h"
 #include "src/tint/sem/type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// A Constant is compile-time known expression value, expressed as a flattened
 /// list of scalar values. Value may be of a scalar or vector type.
@@ -139,7 +138,6 @@ class Constant {
   Scalars elems_;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_CONSTANT_H_

src/tint/sem/depth_multisampled_texture_type.cc

@@ -19,8 +19,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::DepthMultisampledTexture);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 bool IsValidDepthDimension(ast::TextureDimension dim) {
@@ -57,5 +56,4 @@ std::string DepthMultisampledTexture::FriendlyName(const SymbolTable&) const {
   return out.str();
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/depth_multisampled_texture_type.h

@@ -19,8 +19,7 @@
 
 #include "src/tint/sem/texture_type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// A multisampled depth texture type.
 class DepthMultisampledTexture final
@@ -46,7 +45,6 @@ class DepthMultisampledTexture final
   std::string FriendlyName(const SymbolTable& symbols) const override;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_DEPTH_MULTISAMPLED_TEXTURE_TYPE_H_

src/tint/sem/depth_multisampled_texture_type_test.cc

@@ -20,8 +20,7 @@
 #include "src/tint/sem/sampled_texture_type.h"
 #include "src/tint/sem/storage_texture_type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 using DepthMultisampledTextureTest = TestHelper;
@@ -60,5 +59,4 @@ TEST_F(DepthMultisampledTextureTest, FriendlyName) {
 }
 
 }  // namespace
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/depth_texture_type.cc

@@ -19,8 +19,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::DepthTexture);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 bool IsValidDepthDimension(ast::TextureDimension dim) {
@@ -57,5 +56,4 @@ std::string DepthTexture::FriendlyName(const SymbolTable&) const {
   return out.str();
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/depth_texture_type.h

@@ -19,8 +19,7 @@
 
 #include "src/tint/sem/texture_type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// A depth texture type.
 class DepthTexture final : public Castable<DepthTexture, Texture> {
@@ -45,7 +44,6 @@ class DepthTexture final : public Castable<DepthTexture, Texture> {
   std::string FriendlyName(const SymbolTable& symbols) const override;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_DEPTH_TEXTURE_TYPE_H_

src/tint/sem/depth_texture_type_test.cc

@@ -20,8 +20,7 @@
 #include "src/tint/sem/sampled_texture_type.h"
 #include "src/tint/sem/storage_texture_type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 using DepthTextureTest = TestHelper;
@@ -74,5 +73,4 @@ TEST_F(DepthTextureTest, FriendlyName) {
 }
 
 }  // namespace
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/expression.cc

@@ -18,8 +18,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::Expression);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 Expression::Expression(const ast::Expression* declaration,
                        const sem::Type* type,
@@ -36,5 +35,4 @@ Expression::Expression(const ast::Expression* declaration,
 
 Expression::~Expression() = default;
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/expression.h

@@ -20,12 +20,13 @@
 #include "src/tint/sem/constant.h"
 #include "src/tint/sem/node.h"
 
-namespace tint {
-namespace sem {
 // Forward declarations
+namespace tint::sem {
 class Statement;
 class Type;
+}  // namespace tint::sem
 
+namespace tint::sem {
 /// Expression holds the semantic information for expression nodes.
 class Expression : public Castable<Expression, Node> {
  public:
@@ -77,7 +78,6 @@ class Expression : public Castable<Expression, Node> {
   const bool has_side_effects_;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_EXPRESSION_H_

src/tint/sem/external_texture_type.cc

@@ -18,8 +18,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::ExternalTexture);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 ExternalTexture::ExternalTexture() : Base(ast::TextureDimension::k2d) {}
 
@@ -39,5 +38,4 @@ std::string ExternalTexture::FriendlyName(const SymbolTable&) const {
   return "texture_external";
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/external_texture_type.h

@@ -19,8 +19,7 @@
 
 #include "src/tint/sem/texture_type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// An external texture type
 class ExternalTexture final : public Castable<ExternalTexture, Texture> {
@@ -45,7 +44,6 @@ class ExternalTexture final : public Castable<ExternalTexture, Texture> {
   std::string FriendlyName(const SymbolTable& symbols) const override;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_EXTERNAL_TEXTURE_TYPE_H_

src/tint/sem/external_texture_type_test.cc

@@ -20,8 +20,7 @@
 #include "src/tint/sem/storage_texture_type.h"
 #include "src/tint/sem/test_helper.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 using ExternalTextureTest = TestHelper;
@@ -68,5 +67,4 @@ TEST_F(ExternalTextureTest, FriendlyName) {
 }
 
 }  // namespace
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/f32_type.cc

@@ -18,8 +18,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::F32);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 F32::F32() = default;
 
@@ -51,5 +50,4 @@ uint32_t F32::Align() const {
   return 4;
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/f32_type.h

@@ -19,8 +19,7 @@
 
 #include "src/tint/sem/type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// A float 32 type
 class F32 final : public Castable<F32, Type> {
@@ -54,7 +53,6 @@ class F32 final : public Castable<F32, Type> {
   uint32_t Align() const override;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_F32_TYPE_H_

src/tint/sem/f32_type_test.cc

@@ -15,8 +15,7 @@
 #include "src/tint/sem/test_helper.h"
 #include "src/tint/sem/texture_type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 using F32Test = TestHelper;
@@ -46,5 +45,4 @@ TEST_F(F32Test, FriendlyName) {
 }
 
 }  // namespace
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/for_loop_statement.cc

@@ -18,8 +18,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::ForLoopStatement);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 ForLoopStatement::ForLoopStatement(const ast::ForLoopStatement* declaration,
                                    const CompoundStatement* parent,
@@ -32,5 +31,4 @@ const ast::ForLoopStatement* ForLoopStatement::Declaration() const {
   return static_cast<const ast::ForLoopStatement*>(Base::Declaration());
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/for_loop_statement.h

@@ -17,17 +17,17 @@
 
 #include "src/tint/sem/statement.h"
 
+// Forward Declarations
 namespace tint {
 namespace ast {
 class ForLoopStatement;
-}  // namespace ast
+}
 namespace sem {
 class Expression;
-}  // namespace sem
+}
 }  // namespace tint
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// Holds semantic information about a for-loop statement
 class ForLoopStatement final
@@ -58,7 +58,6 @@ class ForLoopStatement final
   const Expression* condition_ = nullptr;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_FOR_LOOP_STATEMENT_H_

src/tint/sem/function.cc

@@ -25,8 +25,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::Function);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 Function::Function(const ast::Function* declaration,
                    Type* return_type,
@@ -38,7 +37,7 @@ Function::Function(const ast::Function* declaration,
   for (auto* parameter : parameters) {
     parameter->SetOwner(this);
   }
-}  // namespace sem
+}
 
 Function::~Function() = default;
 
@@ -193,5 +192,4 @@ Function::TransitivelyReferencedSampledTextureVariablesImpl(
   return ret;
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/function.h

@@ -23,20 +23,21 @@
 #include "src/tint/sem/call.h"
 #include "src/tint/utils/unique_vector.h"
 
-namespace tint {
-
 // Forward declarations
+namespace tint {
 namespace ast {
 class BuiltinAttribute;
 class Function;
 class LocationAttribute;
 class ReturnStatement;
 }  // namespace ast
-
 namespace sem {
-
 class Builtin;
 class Variable;
+}  // namespace sem
+}  // namespace tint
+
+namespace tint::sem {
 
 /// WorkgroupDimension describes the size of a single dimension of an entry
 /// point's workgroup size.
@@ -283,7 +284,6 @@ class Function final : public Castable<Function, CallTarget> {
   sem::Behaviors behaviors_{sem::Behavior::kNext};
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_FUNCTION_H_

src/tint/sem/i32_type.cc

@@ -18,8 +18,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::I32);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 I32::I32() = default;
 
@@ -51,5 +50,4 @@ uint32_t I32::Align() const {
   return 4;
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/i32_type.h

@@ -19,8 +19,7 @@
 
 #include "src/tint/sem/type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// A signed int 32 type.
 class I32 final : public Castable<I32, Type> {
@@ -54,7 +53,6 @@ class I32 final : public Castable<I32, Type> {
   uint32_t Align() const override;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_I32_TYPE_H_

src/tint/sem/i32_type_test.cc

@@ -15,8 +15,7 @@
 #include "src/tint/sem/test_helper.h"
 #include "src/tint/sem/texture_type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 using I32Test = TestHelper;
@@ -46,5 +45,4 @@ TEST_F(I32Test, FriendlyName) {
 }
 
 }  // namespace
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/if_statement.cc

@@ -19,8 +19,7 @@
 TINT_INSTANTIATE_TYPEINFO(tint::sem::IfStatement);
 TINT_INSTANTIATE_TYPEINFO(tint::sem::ElseStatement);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 IfStatement::IfStatement(const ast::IfStatement* declaration,
                          const CompoundStatement* parent,
@@ -44,5 +43,4 @@ const ast::ElseStatement* ElseStatement::Declaration() const {
   return static_cast<const ast::ElseStatement*>(Base::Declaration());
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/if_statement.h

@@ -25,11 +25,10 @@ class ElseStatement;
 }  // namespace ast
 namespace sem {
 class Expression;
-}  // namespace sem
+}
 }  // namespace tint
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// Holds semantic information about an if statement
 class IfStatement final : public Castable<IfStatement, CompoundStatement> {
@@ -92,7 +91,6 @@ class ElseStatement final : public Castable<ElseStatement, CompoundStatement> {
   const Expression* condition_ = nullptr;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_IF_STATEMENT_H_

src/tint/sem/info.cc

@@ -14,8 +14,7 @@
 
 #include "src/tint/sem/info.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 Info::Info() = default;
 
@@ -25,5 +24,4 @@ Info::~Info() = default;
 
 Info& Info::operator=(Info&&) = default;
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/info.h

@@ -22,10 +22,12 @@
 #include "src/tint/sem/node.h"
 #include "src/tint/sem/type_mappings.h"
 
-namespace tint::sem {
-
 // Forward declarations
+namespace tint::sem {
 class Module;
+}
+
+namespace tint::sem {
 
 /// Info holds all the resolved semantic information for a Program.
 class Info {

src/tint/sem/loop_statement.cc

@@ -19,8 +19,7 @@
 TINT_INSTANTIATE_TYPEINFO(tint::sem::LoopStatement);
 TINT_INSTANTIATE_TYPEINFO(tint::sem::LoopContinuingBlockStatement);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 LoopStatement::LoopStatement(const ast::LoopStatement* declaration,
                              const CompoundStatement* parent,
@@ -42,5 +41,4 @@ LoopContinuingBlockStatement::LoopContinuingBlockStatement(
 }
 LoopContinuingBlockStatement::~LoopContinuingBlockStatement() = default;
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/loop_statement.h

@@ -18,14 +18,11 @@
 #include "src/tint/sem/block_statement.h"
 
 // Forward declarations
-namespace tint {
+namespace tint::ast {
-namespace ast {
 class LoopStatement;
-}  // namespace ast
+}
-}  // namespace tint
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// Holds semantic information about a loop statement
 class LoopStatement final : public Castable<LoopStatement, CompoundStatement> {
@@ -58,7 +55,6 @@ class LoopContinuingBlockStatement final
   ~LoopContinuingBlockStatement() override;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_LOOP_STATEMENT_H_

src/tint/sem/matrix_type.cc

@@ -20,8 +20,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::Matrix);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 Matrix::Matrix(const Vector* column_type, uint32_t columns)
     : subtype_(column_type->type()),
@@ -74,5 +73,4 @@ uint32_t Matrix::ColumnStride() const {
   return column_type_->Align();
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/matrix_type.h

@@ -19,11 +19,12 @@
 
 #include "src/tint/sem/type.h"
 
-namespace tint {
-namespace sem {
-
 // Forward declaration
+namespace tint::sem {
 class Vector;
+}
+
+namespace tint::sem {
 
 /// A matrix type
 class Matrix final : public Castable<Matrix, Type> {
@@ -79,7 +80,6 @@ class Matrix final : public Castable<Matrix, Type> {
   const uint32_t columns_;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_MATRIX_TYPE_H_

src/tint/sem/matrix_type_test.cc

@@ -15,8 +15,7 @@
 #include "src/tint/sem/test_helper.h"
 #include "src/tint/sem/texture_type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 using MatrixTest = TestHelper;
@@ -73,5 +72,4 @@ TEST_F(MatrixTest, FriendlyName) {
 }
 
 }  // namespace
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/member_accessor_expression.cc

@@ -21,8 +21,7 @@ TINT_INSTANTIATE_TYPEINFO(tint::sem::MemberAccessorExpression);
 TINT_INSTANTIATE_TYPEINFO(tint::sem::StructMemberAccess);
 TINT_INSTANTIATE_TYPEINFO(tint::sem::Swizzle);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 MemberAccessorExpression::MemberAccessorExpression(
     const ast::MemberAccessorExpression* declaration,
@@ -53,5 +52,4 @@ Swizzle::Swizzle(const ast::MemberAccessorExpression* declaration,
 
 Swizzle::~Swizzle() = default;
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/member_accessor_expression.h

@@ -19,18 +19,18 @@
 
 #include "src/tint/sem/expression.h"
 
-namespace tint {
-
 /// Forward declarations
+namespace tint {
 namespace ast {
 class MemberAccessorExpression;
 }  // namespace ast
-
 namespace sem {
-
-/// Forward declarations
 class Struct;
 class StructMember;
+}  // namespace sem
+}  // namespace tint
+
+namespace tint::sem {
 
 /// MemberAccessorExpression holds the semantic information for a
 /// ast::MemberAccessorExpression node.
@@ -105,7 +105,6 @@ class Swizzle final : public Castable<Swizzle, MemberAccessorExpression> {
   std::vector<uint32_t> const indices_;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_MEMBER_ACCESSOR_EXPRESSION_H_

src/tint/sem/multisampled_texture_type.cc

@@ -19,8 +19,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::MultisampledTexture);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 MultisampledTexture::MultisampledTexture(ast::TextureDimension dim,
                                          const Type* type)
@@ -52,5 +51,4 @@ std::string MultisampledTexture::FriendlyName(
   return out.str();
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/multisampled_texture_type.h

@@ -19,8 +19,7 @@
 
 #include "src/tint/sem/texture_type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// A multisampled texture type.
 class MultisampledTexture final
@@ -53,7 +52,6 @@ class MultisampledTexture final
   const Type* const type_;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_MULTISAMPLED_TEXTURE_TYPE_H_

src/tint/sem/multisampled_texture_type_test.cc

@@ -20,8 +20,7 @@
 #include "src/tint/sem/storage_texture_type.h"
 #include "src/tint/sem/test_helper.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 using MultisampledTextureTest = TestHelper;
@@ -99,5 +98,4 @@ TEST_F(MultisampledTextureTest, FriendlyName) {
 }
 
 }  // namespace
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/node.cc

@@ -16,8 +16,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::Node);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 Node::Node() = default;
 
@@ -25,5 +24,4 @@ Node::Node(const Node&) = default;
 
 Node::~Node() = default;
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/node.h

@@ -17,8 +17,7 @@
 
 #include "src/tint/castable.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// Node is the base class for all semantic nodes
 class Node : public Castable<Node> {
@@ -33,7 +32,6 @@ class Node : public Castable<Node> {
   ~Node() override;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_NODE_H_

src/tint/sem/parameter_usage.cc

@@ -24,8 +24,7 @@
 
 #include "src/tint/sem/parameter_usage.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 const char* str(ParameterUsage usage) {
   switch (usage) {
@@ -61,5 +60,4 @@ const char* str(ParameterUsage usage) {
   return "<unknown>";
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/parameter_usage.cc.tmpl

@@ -10,8 +10,7 @@ See:
 
 #include "src/tint/sem/parameter_usage.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 const char* str(ParameterUsage usage) {
   switch (usage) {
@@ -25,5 +24,4 @@ const char* str(ParameterUsage usage) {
   return "<unknown>";
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/parameter_usage.h

@@ -25,8 +25,7 @@
 #ifndef SRC_TINT_SEM_PARAMETER_USAGE_H_
 #define SRC_TINT_SEM_PARAMETER_USAGE_H_
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// ParameterUsage is extra metadata for identifying a parameter based on its
 /// overload position
@@ -50,7 +49,6 @@ enum class ParameterUsage {
 /// @returns a string representation of the given parameter usage.
 const char* str(ParameterUsage usage);
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_PARAMETER_USAGE_H_

src/tint/sem/parameter_usage.h.tmpl

@@ -11,8 +11,7 @@ See:
 #ifndef SRC_TINT_SEM_PARAMETER_USAGE_H_
 #define SRC_TINT_SEM_PARAMETER_USAGE_H_
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// ParameterUsage is extra metadata for identifying a parameter based on its
 /// overload position
@@ -26,7 +25,6 @@ enum class ParameterUsage {
 /// @returns a string representation of the given parameter usage.
 const char* str(ParameterUsage usage);
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_PARAMETER_USAGE_H_

src/tint/sem/pipeline_stage_set.h

@@ -18,12 +18,10 @@
 #include "src/tint/ast/pipeline_stage.h"
 #include "src/tint/utils/enum_set.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 using PipelineStageSet = utils::EnumSet<ast::PipelineStage>;
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_PIPELINE_STAGE_SET_H_

src/tint/sem/pointer_type.cc

@@ -20,8 +20,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::Pointer);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 Pointer::Pointer(const Type* subtype,
                  ast::StorageClass storage_class,
@@ -59,5 +58,4 @@ Pointer::Pointer(Pointer&&) = default;
 
 Pointer::~Pointer() = default;
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/pointer_type.h

@@ -21,8 +21,7 @@
 #include "src/tint/ast/storage_class.h"
 #include "src/tint/sem/type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// A pointer type.
 class Pointer final : public Castable<Pointer, Type> {
@@ -66,7 +65,6 @@ class Pointer final : public Castable<Pointer, Type> {
   ast::Access const access_;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_POINTER_TYPE_H_

src/tint/sem/pointer_type_test.cc

@@ -15,8 +15,7 @@
 #include "src/tint/sem/test_helper.h"
 #include "src/tint/sem/texture_type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 using PointerTest = TestHelper;
@@ -93,5 +92,4 @@ TEST_F(PointerTest, FriendlyNameWithStorageClass) {
 }
 
 }  // namespace
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/reference_type.cc

@@ -19,8 +19,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::Reference);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 Reference::Reference(const Type* subtype,
                      ast::StorageClass storage_class,
@@ -58,5 +57,4 @@ Reference::Reference(Reference&&) = default;
 
 Reference::~Reference() = default;
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/reference_type.h

@@ -21,8 +21,7 @@
 #include "src/tint/ast/storage_class.h"
 #include "src/tint/sem/type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// A reference type.
 class Reference final : public Castable<Reference, Type> {
@@ -66,7 +65,6 @@ class Reference final : public Castable<Reference, Type> {
   ast::Access const access_;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_REFERENCE_TYPE_H_

src/tint/sem/reference_type_test.cc

@@ -15,8 +15,7 @@
 #include "src/tint/sem/reference_type.h"
 #include "src/tint/sem/test_helper.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 using ReferenceTest = TestHelper;
@@ -93,5 +92,4 @@ TEST_F(ReferenceTest, FriendlyNameWithStorageClass) {
 }
 
 }  // namespace
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/sampled_texture_type.cc

@@ -19,8 +19,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::SampledTexture);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 SampledTexture::SampledTexture(ast::TextureDimension dim, const Type* type)
     : Base(dim), type_(type) {
@@ -49,5 +48,4 @@ std::string SampledTexture::FriendlyName(const SymbolTable& symbols) const {
   return out.str();
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/sampled_texture_type.h

@@ -19,8 +19,7 @@
 
 #include "src/tint/sem/texture_type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// A sampled texture type.
 class SampledTexture final : public Castable<SampledTexture, Texture> {
@@ -52,7 +51,6 @@ class SampledTexture final : public Castable<SampledTexture, Texture> {
   const Type* const type_;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_SAMPLED_TEXTURE_TYPE_H_

src/tint/sem/sampled_texture_type_test.cc

@@ -19,8 +19,7 @@
 #include "src/tint/sem/storage_texture_type.h"
 #include "src/tint/sem/test_helper.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 using SampledTextureTest = TestHelper;
@@ -91,5 +90,4 @@ TEST_F(SampledTextureTest, FriendlyName) {
 }
 
 }  // namespace
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/sampler_texture_pair.h

@@ -20,8 +20,7 @@
 
 #include "src/tint/sem/binding_point.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// Mapping of a sampler to a texture it samples.
 struct SamplerTexturePair {
@@ -46,8 +45,7 @@ struct SamplerTexturePair {
   }
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 namespace std {
 

src/tint/sem/sampler_type.cc

@@ -19,8 +19,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::Sampler);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 Sampler::Sampler(ast::SamplerKind kind) : kind_(kind) {}
 
@@ -43,5 +42,4 @@ std::string Sampler::FriendlyName(const SymbolTable&) const {
   return kind_ == ast::SamplerKind::kSampler ? "sampler" : "sampler_comparison";
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/sampler_type.h

@@ -20,8 +20,7 @@
 #include "src/tint/ast/sampler.h"
 #include "src/tint/sem/type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// A sampler type.
 class Sampler final : public Castable<Sampler, Type> {
@@ -57,7 +56,6 @@ class Sampler final : public Castable<Sampler, Type> {
   ast::SamplerKind const kind_;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_SAMPLER_TYPE_H_

src/tint/sem/sampler_type_test.cc

@@ -16,8 +16,7 @@
 #include "src/tint/sem/test_helper.h"
 #include "src/tint/sem/texture_type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 using SamplerTest = TestHelper;
@@ -67,5 +66,4 @@ TEST_F(SamplerTest, FriendlyNameComparisonSampler) {
 }
 
 }  // namespace
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/sem_array_test.cc

@@ -15,8 +15,7 @@
 #include "src/tint/sem/test_helper.h"
 #include "src/tint/sem/texture_type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 using ArrayTest = TestHelper;
@@ -127,5 +126,4 @@ TEST_F(ArrayTest, FriendlyNameStaticSizedNonImplicitStride) {
 }
 
 }  // namespace
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/sem_struct_test.cc

@@ -16,8 +16,7 @@
 #include "src/tint/sem/test_helper.h"
 #include "src/tint/sem/texture_type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 using StructTest = TestHelper;
@@ -120,5 +119,4 @@ TEST_F(StructTest, Layout) {
 }
 
 }  // namespace
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/statement.cc

@@ -23,8 +23,7 @@
 TINT_INSTANTIATE_TYPEINFO(tint::sem::Statement);
 TINT_INSTANTIATE_TYPEINFO(tint::sem::CompoundStatement);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 Statement::Statement(const ast::Statement* declaration,
                      const CompoundStatement* parent,
@@ -44,5 +43,4 @@ CompoundStatement::CompoundStatement(const ast::Statement* declaration,
 
 CompoundStatement::~CompoundStatement() = default;
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/statement.h

@@ -26,16 +26,12 @@ class Statement;
 }  // namespace ast
 namespace sem {
 class BlockStatement;
+class CompoundStatement;
+class Function;
 }  // namespace sem
 }  // namespace tint
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
-
-/// Forward declaration
-class CompoundStatement;
-class Function;
-
 namespace detail {
 /// FindFirstParentReturn is a traits helper for determining the return type for
 /// the template member function Statement::FindFirstParent().
@@ -182,7 +178,6 @@ const detail::FindFirstParentReturnType<TYPES...>* Statement::FindFirstParent()
   return nullptr;
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_STATEMENT_H_

src/tint/sem/storage_texture_type.cc

@@ -19,8 +19,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::StorageTexture);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 StorageTexture::StorageTexture(ast::TextureDimension dim,
                                ast::TexelFormat format,
@@ -87,5 +86,4 @@ sem::Type* StorageTexture::SubtypeFor(ast::TexelFormat format,
   return nullptr;
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/storage_texture_type.h

@@ -21,10 +21,12 @@
 #include "src/tint/ast/storage_texture.h"
 #include "src/tint/sem/texture_type.h"
 
-namespace tint {
+// Forward Declarations
-namespace sem {
+namespace tint::sem {
-
 class Manager;
+}
+
+namespace tint::sem {
 
 /// A storage texture type.
 class StorageTexture final : public Castable<StorageTexture, Texture> {
@@ -75,7 +77,6 @@ class StorageTexture final : public Castable<StorageTexture, Texture> {
   Type* const subtype_;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_STORAGE_TEXTURE_TYPE_H_

src/tint/sem/storage_texture_type_test.cc

@@ -19,8 +19,7 @@
 #include "src/tint/sem/sampled_texture_type.h"
 #include "src/tint/sem/test_helper.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 struct StorageTextureTest : public TestHelper {
@@ -152,5 +151,4 @@ TEST_F(StorageTextureTest, I32) {
 }
 
 }  // namespace
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/struct.cc

@@ -26,8 +26,7 @@
 TINT_INSTANTIATE_TYPEINFO(tint::sem::Struct);
 TINT_INSTANTIATE_TYPEINFO(tint::sem::StructMember);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 Struct::Struct(const ast::Struct* declaration,
                Symbol name,
@@ -181,5 +180,4 @@ StructMember::StructMember(const ast::StructMember* declaration,
 
 StructMember::~StructMember() = default;
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/struct.h

@@ -27,18 +27,18 @@
 #include "src/tint/sem/type.h"
 #include "src/tint/symbol.h"
 
-namespace tint {
-
 // Forward declarations
+namespace tint {
 namespace ast {
 class StructMember;
 }  // namespace ast
-
 namespace sem {
-
-// Forward declarations
 class StructMember;
 class Type;
+}  // namespace sem
+}  // namespace tint
+
+namespace tint::sem {
 
 /// A vector of StructMember pointers.
 using StructMemberList = std::vector<const StructMember*>;
@@ -230,7 +230,6 @@ class StructMember : public Castable<StructMember, Node> {
   const uint32_t size_;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_STRUCT_H_

src/tint/sem/switch_statement.cc

@@ -19,8 +19,7 @@
 TINT_INSTANTIATE_TYPEINFO(tint::sem::CaseStatement);
 TINT_INSTANTIATE_TYPEINFO(tint::sem::SwitchStatement);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 SwitchStatement::SwitchStatement(const ast::SwitchStatement* declaration,
                                  const CompoundStatement* parent,
@@ -49,5 +48,4 @@ const ast::CaseStatement* CaseStatement::Declaration() const {
   return static_cast<const ast::CaseStatement*>(Base::Declaration());
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/switch_statement.h

@@ -18,15 +18,12 @@
 #include "src/tint/sem/block_statement.h"
 
 // Forward declarations
-namespace tint {
+namespace tint::ast {
-namespace ast {
 class CaseStatement;
 class SwitchStatement;
-}  // namespace ast
+}  // namespace tint::ast
-}  // namespace tint
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// Holds semantic information about an switch statement
 class SwitchStatement final
@@ -74,7 +71,6 @@ class CaseStatement final : public Castable<CaseStatement, CompoundStatement> {
   const BlockStatement* body_ = nullptr;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_SWITCH_STATEMENT_H_

src/tint/sem/test_helper.h

@@ -20,8 +20,7 @@
 #include "gtest/gtest.h"
 #include "src/tint/program_builder.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// Helper class for testing
 template <typename BASE>
@@ -43,7 +42,6 @@ using TestHelper = TestHelperBase<testing::Test>;
 template <typename T>
 using TestParamHelper = TestHelperBase<testing::TestWithParam<T>>;
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_TEST_HELPER_H_

src/tint/sem/texture_type.cc

@@ -16,8 +16,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::Texture);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 Texture::Texture(ast::TextureDimension dim) : dim_(dim) {}
 
@@ -25,5 +24,4 @@ Texture::Texture(Texture&&) = default;
 
 Texture::~Texture() = default;
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/texture_type.h

@@ -18,8 +18,7 @@
 #include "src/tint/ast/texture.h"
 #include "src/tint/sem/type.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// A texture type.
 class Texture : public Castable<Texture, Type> {
@@ -38,7 +37,6 @@ class Texture : public Castable<Texture, Type> {
   ast::TextureDimension const dim_;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_TEXTURE_TYPE_H_

src/tint/sem/texture_type_test.cc

@@ -17,8 +17,7 @@
 #include "src/tint/sem/sampled_texture_type.h"
 #include "src/tint/sem/test_helper.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 namespace {
 
 using TextureTypeDimTest = TestParamHelper<ast::TextureDimension>;
@@ -42,5 +41,4 @@ INSTANTIATE_TEST_SUITE_P(Dimensions,
                                            ast::TextureDimension::kCubeArray));
 
 }  // namespace
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/type.cc

@@ -27,8 +27,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::Type);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 Type::Type() = default;
 
@@ -154,5 +153,4 @@ bool Type::is_handle() const {
   return IsAnyOf<Sampler, Texture>();
 }
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/type.h

@@ -20,13 +20,13 @@
 
 #include "src/tint/sem/node.h"
 
-namespace tint {
-
 // Forward declarations
+namespace tint {
 class ProgramBuilder;
 class SymbolTable;
+}  // namespace tint
 
-namespace sem {
+namespace tint::sem {
 
 /// Supported memory layouts for calculating sizes
 enum class MemoryLayout { kUniformBuffer, kStorageBuffer };
@@ -118,8 +118,7 @@ class Type : public Castable<Type, Node> {
   Type();
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 namespace std {
 

src/tint/sem/type_constructor.cc

@@ -16,8 +16,7 @@
 
 TINT_INSTANTIATE_TYPEINFO(tint::sem::TypeConstructor);
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 TypeConstructor::TypeConstructor(const sem::Type* type,
                                  const ParameterList& parameters)
@@ -25,5 +24,4 @@ TypeConstructor::TypeConstructor(const sem::Type* type,
 
 TypeConstructor::~TypeConstructor() = default;
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint

src/tint/sem/type_constructor.h

@@ -17,8 +17,7 @@
 
 #include "src/tint/sem/call_target.h"
 
-namespace tint {
+namespace tint::sem {
-namespace sem {
 
 /// TypeConstructor is the CallTarget for a type constructor.
 class TypeConstructor final : public Castable<TypeConstructor, CallTarget> {
@@ -32,7 +31,6 @@ class TypeConstructor final : public Castable<TypeConstructor, CallTarget> {
   ~TypeConstructor() override;
 };
 
-}  // namespace sem
+}  // namespace tint::sem
-}  // namespace tint
 
 #endif  // SRC_TINT_SEM_TYPE_CONSTRUCTOR_H_