review

Author: danakj

toolchain/check/facet_type.cpp

@@ -55,7 +55,7 @@ static auto WitnessQueryMatchesInterface(
 static auto IncompleteFacetTypeDiagnosticBuilder(
     Context& context, SemIR::LocId loc_id, SemIR::TypeInstId facet_type_inst_id,
     bool is_definition) -> DiagnosticBuilder {
-  // TODO: Remove this parameter. Facet types don't need to be complete for in
+  // TODO: Remove this parameter. Facet types don't need to be complete for impl
   // declarations, unless there's a rewrite into `.Self`. But that completeness
   // is checked/required by the member access of the rewrite.
   CARBON_CHECK(is_definition);

toolchain/check/handle_require.cpp

@@ -291,7 +291,6 @@ auto HandleParseNode(Context& context, Parse::RequireDeclId node_id) -> bool {
                                              RequireImplsIncompleteFacetType,
                                              constraint_inst_id);
             })) {
-      // DiscardGenericDecl(context);
       return true;
     }
   }