[PATCH] D86319: [Attributor] Introduce module slice.

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Aug 29 14:15:07 PDT 2020


jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.
Herald added a subscriber: danielkiss.

LGTM, one nit



================
Comment at: llvm/include/llvm/Transforms/IPO/Attributor.h:1084
     if (FnScope && !Functions.count(const_cast<Function *>(FnScope))) {
-      AA.getState().indicatePessimisticFixpoint();
-      return AA;
+      if (!getInfoCache().ModuleSlice.count(const_cast<Function *>(FnScope))) {
+        AA.getState().indicatePessimisticFixpoint();
----------------
make this a proper query method, isInModuleSlice or something.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86319



More information about the llvm-commits mailing list