search for: pr3913

Hello, This (quite big :-)) patch fixes PR3913. Regards -- Jakub Staszak -------------- next part -------------- A non-text attachment was scrubbed... Name: pr3913.patch Type: application/octet-stream Size: 1451 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20090824/e9d36629/attachment.obj>

Looks good. Pl. apply. Thanks, - Devang On Mon, Aug 24, 2009 at 2:29 PM, Jakub Staszak<kuba at gcc.gnu.org> wrote: > Hello, > > This (quite big :-)) patch fixes PR3913. > > Regards > -- > Jakub Staszak > _______________________________________________ > LLVM Developers mailing list > LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev > > -- - Devang

On Aug 25, 2009, at 3:11 PM, Devang Patel wrote: > Looks good. Pl. apply. Looks good to me, but please change the testcase to not use "lli" to validate the test. "Execution" tests should be very rare. -Chris

...; >> Looks good. Pl. apply. > > Looks good to me, but please change the testcase to not use "lli" to > validate the test. "Execution" tests should be very rare. > > -Chris -------------- next part -------------- A non-text attachment was scrubbed... Name: pr3913-2.patch Type: application/octet-stream Size: 1428 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20090827/db9cb4bf/attachment.obj>

...fails on some tests, i.e: test/Transforms/LoopIndexSplit/SaveLastValue-2007-08-17.ll because of PHI (isSafeToSpeculativelyExecute == false). >> It seems that LoopIndexSplit doesn't handle some cases, I'll try to >> send >> some patch this week. > > You mean like PR3913? Or are you referring to something else? 3913 is a bug. I meant some other cases to optimize. However I'll try to look at it as well. Regards -- Jakub Staszak

Hello, This patch fixes PR4174. Two test-cases included: original one from bugzilla and a little bit complicated made be myself. It seems that LoopIndexSplit doesn't handle some cases, I'll try to send some patch this week. Regards -- Jakub Staszak -------------- next part -------------- A non-text attachment was scrubbed... Name: pr4174.patch Type: application/octet-stream Size: