[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