[LLVMbugs] [Bug 14219] LoopIdiomRecognize crashes when using DependenceAnalysis

[bugzilla-daemon at llvm.org]

http://llvm.org/bugs/show_bug.cgi?id=14219

Benjamin Kramer <benny.kra at gmail.com> changed:

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

--- Comment #6 from Benjamin Kramer <benny.kra at gmail.com> 2012-10-31 04:26:08 CDT ---
(In reply to comment #5)
> DepAnalysis is both really new and pretty big/scary/unproven.  Given that LLVM
> 3.2 will be branching soon, if it turns out that relying on it is causing
> serious issues, we should switch loop idiom back, and turn on DepAnalysis after
> the branch.  If this is an obvious bug, then ignore this comment :-)

This is an obvious one, fixed in r167110.

I'm more afraid of complicated bugs like #14214 where the new analysis uses
SCEV in a way that uncovers a lingering cache invalidation in another pass and
creates invalid code. We can decide to roll back to the old analysis when 3.2
has branches and there are still issues.

-- 
Configure bugmail: http://llvm.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.