From c2a0ee98c5bdfd2ab3f4f1fe193f44d2eb784963 Mon Sep 17 00:00:00 2001 From: David Blaikie Date: Thu, 17 Jul 2025 14:32:36 -0400 Subject: [PATCH] Remove outdated comment about generic vtable inst naming (#5814) Since vtables are generic over the class's specific, they don't have their own generic/specific ids and so there's no generic insts that need naming. --- toolchain/sem_ir/inst_namer.cpp | 2 -- 1 file changed, 2 deletions(-) diff --git a/toolchain/sem_ir/inst_namer.cpp b/toolchain/sem_ir/inst_namer.cpp index 4dddab587237..46c3f7d5121c 100644 --- a/toolchain/sem_ir/inst_namer.cpp +++ b/toolchain/sem_ir/inst_namer.cpp @@ -134,8 +134,6 @@ InstNamer::InstNamer(const File* sem_ir) : sem_ir_(sem_ir) { GetScopeInfo(vtable_scope).name = globals_.AllocateName( *this, vtable_loc, sem_ir->names().GetIRBaseName(class_info.name_id).str() + ".vtable"); - // TODO: Add support for generic vtables here and elsewhere. - // CollectNamesInGeneric(vtable_scope, vtable_info.generic_id); } // Build each interface scope.