[PATCH] D76378: [Attributor] Make use of analysis in the MustBeExecutedExplorer

Hideto Ueno via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 19 16:28:02 PDT 2020


uenoku requested changes to this revision.
uenoku added inline comments.
This revision now requires changes to proceed.


================
Comment at: llvm/test/Transforms/Attributor/dereferenceable-2.ll:400
+  ret i32 1
+}
----------------
baziotis wrote:
> jdoerfert wrote:
> > TBH, I'm confused why this works. this test is only run in the old pass manager which doesn't support querying analysis passes (from the Attributor). Where am I wrong?
> That makes sense. I tested locally and it doesn't pass, maybe I miss something too though.
Sorry, I have missed this test uses OPM. It doesn't pass locally. 


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

https://reviews.llvm.org/D76378





More information about the llvm-commits mailing list