[llvm-bugs] [Bug 48150] [LoopLoadElim] Assertion `L->getLoopPreheader() && "Can't expand add recurrences without a loop preheader!"' failed

[via llvm-bugs]

https://bugs.llvm.org/show_bug.cgi?id=48150

Max Kazantsev <max.kazantsev at azul.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #4 from Max Kazantsev <max.kazantsev at azul.com> ---
Fix merged.

commit 664e1da485d244325947d056a271c3c733ad4c7c
Author: Max Kazantsev <mkazantsev at azul.com>
Date:   Thu Nov 26 10:49:44 2020 +0700

    [LoopLoadElim] Make sure all loops are in simplify form. PR48150

    LoopLoadElim may end up expanding an AddRec from a loop
    which is not the current loop. This loop may not be in simplify
    form. We figure it out after the no-return point, so cannot bail
    in this case.

    AddRec requires simplify form to expand. The only way to ensure
    this does not crash is to simplify all loops beforehand.

    The issue only exists in new PM. Old PM requests LoopSimplify
    required pass and it simplifies all loops before the opt begins.

    Differential Revision: https://reviews.llvm.org/D91525
    Reviewed By: asbirlea, aeubanks

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20201126/666e8005/attachment-0001.html>