[llvm-commits] [llvm] r84637 - in /llvm/trunk: lib/Transforms/Scalar/SCCP.cpp test/Transforms/SCCP/2009-06-03-sccp-structret-undef.ll

[Chris Lattner]

On Oct 20, 2009, at 8:15 AM, Torok Edwin wrote:

> Author: edwin
> Date: Tue Oct 20 10:15:09 2009
> New Revision: 84637
>
> URL: http://llvm.org/viewvc/llvm-project?rev=84637&view=rev
> Log:
> Fix PR4313: IPSCCP was not setting the lattice value for the invoke  
> instruction
> when the invoke had multiple return values: it set the lattice value  
> only on the
> extractvalue.
> This caused the invoke's lattice value to remain the default  
> (undefined), and
> later propagated to extractvalue's operand, which incorrectly  
> introduces
> undefined behavior.

Hi Edwin,

Your patch didn't affect the testcase you committed.  I removed the  
test and added a new testcase in r84676.  Thanks for tracking down  
this bug!

-Chris