[PATCH] D62607: LoopDistribute/LAA: Respect convergent

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 29 09:56:30 PDT 2019


jdoerfert added inline comments.


================
Comment at: lib/Analysis/LoopAccessAnalysis.cpp:1783
+  // At first assume there are.
+  CanInsertRuntimeCheck = false;
+
----------------
I would have assumed this to be true initially and set to false if a convergent call was found. That would also remove one of the variables. If there is a reason agains it, please explain.


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

https://reviews.llvm.org/D62607





More information about the llvm-commits mailing list