[PATCH] D98263: [LoopInterchange] fix tightlyNested() in LoopInterchange legality

Congzhe Cao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 10 09:10:33 PST 2021


congzhe added a comment.

In D98263#2616699 <https://reviews.llvm.org/D98263#2616699>, @Whitney wrote:

> Thanks for fixing it. Is the crashing test case an existing test case, or will be added in D96708 <https://reviews.llvm.org/D96708> or D91682 <https://reviews.llvm.org/D91682>? If not, can you please also include that test case in this patch?

Thanks for the reply! The crashing test case is not an existing test case, and I just added it to this patch. To summarize, three test cases can be handled by this patch which could not be handled by trunk code: one is what I just added, the other two are from D96708 <https://reviews.llvm.org/D96708> and D91682 <https://reviews.llvm.org/D91682> as I included in the summary section of this patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98263



More information about the llvm-commits mailing list