[PATCH] D75590: [Attributor] IPO across definition boundary of a function marked alwaysinline

Stefanos Baziotis via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 6 15:29:43 PST 2020


baziotis added inline comments.


================
Comment at: llvm/include/llvm/Transforms/IPO/Attributor.h:1105
+    if (FnScope && FnScope->hasFnAttribute(Attribute::AlwaysInline))
+      if (InfoCache.InlineableFunctions.find(FnScope) ==
+          InfoCache.InlineableFunctions.end())
----------------
Nit: You can use `!InfoCache.InlineableFunctions.count(FnScope)` (it is shorter)


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

https://reviews.llvm.org/D75590





More information about the llvm-commits mailing list