[llvm-commits] [llvm] r128782 - in /llvm/trunk: include/llvm/Transforms/Utils/Local.h lib/Transforms/Utils/Local.cpp test/Transforms/JumpThreading/2011-04-02-SimplifyDeadBlock.ll

[Eli Friedman]

On Sun, Apr 3, 2011 at 4:29 AM, Francois Pichet <pichet2000 at gmail.com> wrote:
> On Sat, Apr 2, 2011 at 6:45 PM, Eli Friedman <eli.friedman at gmail.com> wrote:
>> Author: efriedma
>> Date: Sat Apr  2 17:45:17 2011
>> New Revision: 128782
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=128782&view=rev
>> Log:
>> PR9446: RecursivelyDeleteTriviallyDeadInstructions can delete the instruction
>> after the given instruction; make sure to handle that case correctly.
>> (It's difficult to trigger; the included testcase involves a dead
>> block, but I don't think that's a requirement.)
>>
>> While I'm here, get rid of the unnecessary warning about
>> SimplifyInstructionsInBlock, since it should work correctly as far as I know.
>>
>>
>> Added:
>>    llvm/trunk/test/Transforms/JumpThreading/2011-04-02-SimplifyDeadBlock.ll
>> Modified:
>>    llvm/trunk/include/llvm/Transforms/Utils/Local.h
>>    llvm/trunk/lib/Transforms/Utils/Local.cpp
>
> hi,
>
> This commit is making 3 clang tests to crash on Win32 using MSVC 2010.
> any ideas?
>
> 4>  ********************
> 4>  Failing Tests (3):
> 4>      Clang :: CodeGenCXX/temp-order.cpp
> 4>      Clang :: CodeGenObjC/exceptions.m
> 4>      Clang :: CodeGenObjC/synchronized.m

No idea off the top of my head... this change should only have an
effect in cases where we would have accessed freed memory otherwise.

-Eli