[PATCH] D65825: [Attributor] Introduce checkForAllReadWriteInstructions(...).

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

One small request, otherwise LGTM.



================
Comment at: llvm/include/llvm/Transforms/IPO/Attributor.h:300
+      const Function &F, const llvm::function_ref<bool(Instruction *)> &Pred,
+      AbstractAttribute &QueryingAA, InformationCache &InfoCache);
+
----------------
`bool(Instruction &)`


================
Comment at: llvm/lib/Transforms/IPO/Attributor.cpp:944
-    if (LivenessAA && LivenessAA->isAssumedDead(I))
-      continue;
 
----------------
It's much nice now that the concrete AAs don't need to deal with liveness and the infocache etc. but still get the benefits.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65825