[PATCH] D28482: [loop-unroll] Properly populate LoopInfo for loops cloned in LoopUnrollRuntime.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 9 14:15:12 PST 2017


fhahn created this revision.
fhahn added reviewers: dexonsmith, mzolotukhin.
fhahn added a subscriber: llvm-commits.

This patch updates both LoopUnroll.cpp and LoopUnrollRuntime.cpp to use
the same function to add cloned basic blocks of loops to LoopInfo.

This fixes Transforms/LoopUnroll/runtime-loop3.ll which failed with
EXTENSIVE_DEBUG, because the cloned basic blocks were not added to the
correct sub-loops in LoopUnrollRuntime.cpp.

I suspect LoopUnroll.cpp and LoopUnrollRuntime.cpp could share more code
related to cloning basic blocks of a loop and I could look into that in
follow-up patches if people think this would make sense.


https://reviews.llvm.org/D28482

Files:
  include/llvm/Transforms/Utils/UnrollLoop.h
  lib/Transforms/Utils/LoopUnroll.cpp
  lib/Transforms/Utils/LoopUnrollRuntime.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D28482.83696.patch
Type: text/x-patch
Size: 5303 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170109/6d782da7/attachment.bin>


More information about the llvm-commits mailing list