[PATCH] D111611: [NFC] [LoopPeel] Change the way DT is updated for loop exits

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 12 01:01:07 PDT 2021


fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/LoopPeel.cpp:753
     cloneLoopBlocks(L, Iter, InsertTop, InsertBot, ExitEdges, NewBlocks,
                     LoopBlocks, VMap, LVMap, DT, LI,
                     LoopLocalNoAliasDeclScopes);
----------------
Passing `DT` means that `cloneLoopBlocks` previously could rely on `DT` being up to date here. That's not true any more, as it is only updated later. Is that a problem? We also need to check all other uses of plain `DT` in until the updates are applied.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111611



More information about the llvm-commits mailing list