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

Hideto Ueno via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 4 05:06:35 PST 2020


uenoku added inline comments.


================
Comment at: llvm/include/llvm/Transforms/IPO/Attributor.h:1717
+    if (IsFnInterface && (!FnScope || !FnScope->hasExactDefinition())) {
+      if (FnScope->hasFnAttribute(Attribute::AlwaysInline)) {
+        // check if it is OK to inline, if so, do fix-point analysis for the
----------------
Is `FnScope` always nonnull here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75590





More information about the llvm-commits mailing list