================
@@ -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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to