search for: 0158029c

Hello, Simply removing that "return true" causes the code to search blocks outside of loops for side effects. That's not what the code is supposed to do. Dan On Nov 27, 2009, at 3:01 AM, Jakub Staszak wrote: > Hi, > > Because of this "return true" not every block was visited and only one ExitBB was found (instead of two). Thus, loop was optimized as a trivial

Hi, Because of this "return true" not every block was visited and only one ExitBB was found (instead of two). Thus, loop was optimized as a trivial one, which was wrong. -Jakub On Nov 24, 2009, at 2:28 PM, Dan Gohman wrote: > Hello, > > I haven't studied this in detail, but at a first look this makes the > code inconsistent with the associated comments. Why

...d it out :) Dan, can you look at this patch? Thanks -Jakub -------------- next part -------------- A non-text attachment was scrubbed... Name: pr5373-2.patch Type: application/octet-stream Size: 1672 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20091206/0158029c/attachment.obj> -------------- next part -------------- On Nov 30, 2009, at 7:52 PM, Dan Gohman wrote: > Hello, > > Simply removing that "return true" causes the code to search > blocks outside of loops for side effects. That's not > what the code is supposed to d...