[llvm-commits] [llvm] r133004 - /llvm/trunk/lib/CodeGen/AsmPrinter/AsmPrinter.cpp

[Eli Friedman]

On Tue, Jun 14, 2011 at 12:13 PM, Eli Friedman <eli.friedman at gmail.com> wrote:
> On Tue, Jun 14, 2011 at 11:12 AM, Rafael Espindola
> <rafael.espindola at gmail.com> wrote:
>> Author: rafael
>> Date: Tue Jun 14 13:12:31 2011
>> New Revision: 133004
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=133004&view=rev
>> Log:
>> Partial revert of 132882.
>> Dan noted that this would work on the case shown on the commit message. I think
>> the case that was failing was a bb ending with a redundant conditional jump:
>>
>> ...
>> jne foo
>>
>> foo:
>> ...
>>
>> I was unable to find any such case in the tests or in a debug build of clang,
>> so I will revert this part of the patch and watch the bots.
>
> This is making stuff explode; try
> test-suite/SingleSource/Benchmarks/Adobe-C++/stepanov_vector.cpp , for
> example.

Reverted in r133007.

-Eli