[PATCH] D119878: [FuncSpec] Remove definitions of fully specialized functions.

Alexandros Lamprineas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 16 09:45:08 PST 2022


labrinea added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/FunctionSpecialization.cpp:511
+        all_of(AI.Fn->users(), [&AI](User *U) {
+          if (auto *CS = dyn_cast_or_null<CallBase>(U))
+            return CS->getFunction() == AI.Fn;
----------------
nikic wrote:
> or_null is unnecessary here, user cannot be null.
thanks, will update it accordingly


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119878



More information about the llvm-commits mailing list