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

Wed Mar 10 18:44:07 PST 2021

congzhe added a comment.

In D98263#2617162 <https://reviews.llvm.org/D98263#2617162>, @TaWeiTu wrote:

> In D98263#2617156 <https://reviews.llvm.org/D98263#2617156>, @Whitney wrote:
>
>> The patch now revert part of https://reviews.llvm.org/rGdf9158c9a45a6902c2b0394f9bd6512e3e441f31, which leave some changes behind.
>> Given that the change cause a test case crash, @TaWeiTu can you please revert the change?
>
> Sure, reverted.

Thanks a lot for the update!

I updated my patch accordingly based on the latest trunk code. Now we do not have `pr48113.ll` which was in D97290 <https://reviews.llvm.org/D97290>. If necessary maybe @TaWeiTu or I could add it back. `pr48113.ll` is handled in this patch.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98263