[PATCH] D15504: Bug fix for shrink-wrap

[Quentin Colombet via llvm-commits]

Thanks!

I’ll clean-up the patch and push with your additional fix.

Q.
> On Dec 14, 2015, at 4:11 PM, Weiming Zhao <weimingz at codeaurora.org> wrote:
> 
> weimingz added a comment.
> 
> Hi Quentin,
> 
> Yes, but I need to change one line because otherwise, it will crash with other inputs.
> 
> else ==>  else if (MLI->getLoopFor(Restore)) {
> 
>  // If the loop does not exit, ....
>  SmallVector<...> ExitBlocks;
> 
> MLI->getLoopFor(Restore)->getExitingBlocks(ExitBlocks);  ==> need to check if Restore is in loop
> }
> 
> In http://reviews.llvm.org/D15504#310456, @qcolombet wrote:
> 
>> Hi Weiming,
>> 
>> Could you try if the attached patch fixes the problem?
>> If it does, that should be a much safer workaround.
>> 
>> Thanks,
>> -Quentin
>> 
>> - F1174870: shrink-post-loop.patch <http://reviews.llvm.org/F1174870>
>> - F1174869: msg-28761-15.txt <http://reviews.llvm.org/F1174869>
> 
> 
> 
> 
> 
> Repository:
>  rL LLVM
> 
> http://reviews.llvm.org/D15504
> 
> 
>