[PATCH] D135968: [FuncSpec][NFC] Refactor finding specialisation opportunities

Alexandros Lamprineas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 14 09:44:36 PDT 2022


labrinea accepted this revision.
labrinea added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Transforms/IPO/FunctionSpecialization.cpp:466
 
-        if (I.second)
-          S.Gain = ForceFunctionSpecialization ? 1 : 0 - Cost;
+        SpecializationInfo &S = Specializations.back().second;
         if (!ForceFunctionSpecialization)
----------------
This confused me a bit and then I remembered it's a MapVector. LGTM!


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

https://reviews.llvm.org/D135968



More information about the llvm-commits mailing list