Remove unnecessary llvm::formatv() call in TODO(). (#6099)

bricknerb · web-flow · commit 8cbf289c912d · 2025-09-19T09:14:41.000Z
diff --git a/toolchain/check/cpp/import.cpp b/toolchain/check/cpp/import.cpp
@@ -2088,9 +2088,7 @@ static auto GetOverloadSetAccess(Context& context, SemIR::LocId loc_id,
   clang::AccessSpecifier access = overload_set.begin().getAccess();
   for (auto it = overload_set.begin() + 1; it != overload_set.end(); ++it) {
     if (it.getAccess() != access) {
-      context.TODO(
-          loc_id,
-          llvm::formatv("Unsupported: Overloaded set with mixed access").str());
+      context.TODO(loc_id, "Unsupported: Overloaded set with mixed access");
       return std::nullopt;
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -2088,9 +2088,7 @@ static auto GetOverloadSetAccess(Context& context, SemIR::LocId loc_id,`
`2088`	`2088`	`clang::AccessSpecifier access = overload_set.begin().getAccess();`
`2089`	`2089`	`for (auto it = overload_set.begin() + 1; it != overload_set.end(); ++it) {`
`2090`	`2090`	`if (it.getAccess() != access) {`
`2091`		`- context.TODO(`
`2092`		`- loc_id,`
`2093`		`- llvm::formatv("Unsupported: Overloaded set with mixed access").str());`
	`2091`	`+ context.TODO(loc_id, "Unsupported: Overloaded set with mixed access");`
`2094`	`2092`	`return std::nullopt;`
`2095`	`2093`	`}`
`2096`	`2094`	`}`