[llvm] r198456 - Add a LLVM_DUMP_METHOD macro.

[Alp Toker]

On 04/01/2014 15:03, Renato Golin wrote:
> On 3 January 2014 22:53, Nico Weber <nicolasweber at gmx.de 
> <mailto:nicolasweber at gmx.de>> wrote:
>
>     Author: nico
>     Date: Fri Jan  3 16:53:37 2014
>     New Revision: 198456
>
>     URL: http://llvm.org/viewvc/llvm-project?rev=198456&view=rev
>     Log:
>     Add a LLVM_DUMP_METHOD macro.
>
>
>
> Hi Nico,
>
> I'm not sure this is the commit that broke our test-suite buildbot, 
> but it's the only non-trivial patch on an optimization pass that could 
> explain the crash:
>
> http://lab.llvm.org:8011/builders/clang-native-arm-lnt/builds/4438

Hi Renato

I wrote this patch (modulo the "METHOD" naming, sigh) and it's not to blame.

The true culprit was r198478:

     Fix PR18361: Invalidate LoopDispositions after LoopSimplify hoists 
things.

     getSCEV for an ashr instruction creates an intermediate zext
     expression when it truncates its operand.

Reverted in r198503 until Andy gets a chance to look into it.

Alp.



>
> details here:
>
> http://llvm.org/bugs/show_bug.cgi?id=18384
>
> cheers,
> --renato
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

-- 
http://www.nuanti.com
the browser experts