[PATCH] D24716: [Polly] DeLICM/DePRE (WIP)

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 4 08:20:17 PST 2017


Meinersbur added a comment.

In https://reviews.llvm.org/D24716#666573, @grosser wrote:

> >> Do you have an example that makes clear why you need zones, and cannot just track the values / life ranges? That seems like an interesting test for isConflicting.
> > 
> > See file comment in DeLICM.cpp
>
> Could such an example be added to the tests of isConflicting that illustrates why such a merge would result in an incorrect / unprecise answer.


Could you suggest an `EXPECT(...)` line? I have no clue what kind of example you have in mind.


https://reviews.llvm.org/D24716





More information about the llvm-commits mailing list