[PATCH] D62607: LoopDistribute/LAA: Respect convergent
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 29 10:16:48 PDT 2019
arsenm marked an inline comment as done.
arsenm added inline comments.
Comment at: lib/Analysis/LoopAccessAnalysis.cpp:1977
- dbgs() << "LAA: We can perform a memory runtime check if needed.\n");
+ dbgs() << "LAA: May be able to perform a memory runtime check if needed.\n");
> This message carries no information. Remove entirely/move the check from line 2012 before this? The entire code after this seems useless when a convergent call has been found anyway.
It's not useless due to the fact that LoopDistribute barely cares about the final result computed here. It still depends on the analyses done later
CHANGES SINCE LAST ACTION
More information about the llvm-commits