diff --git a/src/ast/literal.h b/src/ast/literal.h
index 783a0f748a..7ae091cb9a 100644
--- a/src/ast/literal.h
+++ b/src/ast/literal.h
@@ -61,6 +61,7 @@ class Literal {
 
  protected:
   /// Constructor
+  /// @param type the type of the literal
   explicit Literal(ast::type::Type* type);
 
  private:
diff --git a/src/context.h b/src/context.h
index f0495f2599..f1117f521b 100644
--- a/src/context.h
+++ b/src/context.h
@@ -30,6 +30,7 @@ class Context {
   /// Resets the state of this context.
   void Reset();
 
+  /// @returns the Type Manager
   TypeManager& type_mgr() { return type_mgr_; }
 
  private:
diff --git a/src/writer/spirv/operand.h b/src/writer/spirv/operand.h
index de66a2265c..844044da9e 100644
--- a/src/writer/spirv/operand.h
+++ b/src/writer/spirv/operand.h
@@ -50,7 +50,9 @@ class Operand {
   ~Operand();
 
   /// Copy assignment
-  Operand& operator=(const Operand&) = default;
+  /// @param b the operand to copy
+  /// @returns a copy of this operand
+  Operand& operator=(const Operand& b) = default;
 
   /// Sets the float value
   /// @param val the value to set