[clang] [llvm] [Coroutines] Change `llvm.coro.noop` to accept `llvm_anyptr_ty` instead (PR #102096)

Matt Arsenault via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 16 06:07:32 PDT 2024


================
@@ -123,6 +123,17 @@ bool coro::declaresIntrinsics(const Module &M,
   return false;
 }
 
+// Verifies if a module has any intrinsics.
+bool coro::declaresIntrinsics(const Module &M,
+                              const DenseSet<Intrinsic::ID> &Identifiers) {
+  for (const Function &F : M.functions()) {
+    if (Identifiers.contains(F.getIntrinsicID()))
----------------
arsenm wrote:

But it's doing it with this intermediate set. As in, this is an overly fancy, heavy API. I mean literally loop over function declarations with a switch in it, no DenseSet 

https://github.com/llvm/llvm-project/pull/102096


More information about the cfe-commits mailing list