[PATCH] D45970: [LoopInterchange] Do not change LI for BBs in child loops.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 23 14:07:19 PDT 2018


fhahn added a comment.

In https://reviews.llvm.org/D45970#1075854, @efriedma wrote:

> LGTM
>
> I'm not really happy with LoopInterchangeTransform::restructureLoops overall; it seems like complicated, fragile code.  But I don't see a better alternative.


Thanks Eli.  It is not ideal.... Maybe there's potential for a more generic API to update LoopInfo, but the things we need to update here seem quite specific to interchanging.


Repository:
  rL LLVM

https://reviews.llvm.org/D45970





More information about the llvm-commits mailing list