[PATCH] D66914: [Attributor] Look at internal functions only on-demand

[Hideto Ueno via Phabricator via llvm-commits]

uenoku added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/Attributor.cpp:144-155
+struct AADummy : public StateWrapper<BooleanState, AbstractAttribute>,
+                 public IRPosition {
+  AADummy(const IRPosition &IRP) : IRPosition(IRP) {}
+  IRPosition &getIRPosition() override { return *this; }
+  const IRPosition &getIRPosition() const override { return *this; }
+  const std::string getAsStr() const override { return ""; }
+  void trackStatistics() const override {}
----------------
Is `AADummy` used in this patch? What is the purpose?


================
Comment at: llvm/test/Transforms/FunctionAttrs/align.ll:1
-; RUN: opt -attributor -attributor-disable=false -attributor-max-iterations-verify -attributor-max-iterations=6 -S < %s | FileCheck %s --check-prefix=ATTRIBUTOR
+; RUN: opt -attributor -attributor-disable=false -attributor-max-iterations-verify -attributor-max-iterations=7 -S < %s | FileCheck %s --check-prefix=ATTRIBUTOR
 
----------------
Why does max-iterations increase? 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66914