[PATCH] D114917: [LoopInterchange] Enable loop interchange with multiple inner loop indvars

Bardia Mahjour via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 14 07:01:48 PST 2022


bmahjour added a comment.

In D114917#3242112 <https://reviews.llvm.org/D114917#3242112>, @congzhe wrote:

> Removed a function `getInductionVariable()` that becomes unused, in order to avoid build bot sanitizer failures.
>
> `getInductionVariable()` becomes unused since we now use `getInnerLoopInductions()` to return the vector of inner indvars, instead of returning a single inner indvar from `getInductionVariable()`.
>
> Does it look fine to you? @bmahjour

Yep, makes sense to remove it.


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

https://reviews.llvm.org/D114917



More information about the llvm-commits mailing list