[PATCH] D119880: [FuncSpec] Support function specialization across multiple arguments.

Alexandros Lamprineas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 16 03:47:03 PDT 2022


labrinea added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/FunctionSpecialization.cpp:447
 
-      // FIXME: Only one argument per function.
-      break;
+      Specializations.remove_if([&Specializations](const auto &Entry) {
+        auto Iter = Specializations.find(Entry.first);
----------------
This looks suboptimal. Will try and write an erase method for MapVector which takes an iterator range.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D119880/new/

https://reviews.llvm.org/D119880



More information about the llvm-commits mailing list