[llvm-commits] [llvm] r70557 - in /llvm/trunk: include/llvm/Analysis/ScalarEvolutionExpander.h lib/Analysis/ScalarEvolutionExpander.cpp

[Dan Gohman]

On May 1, 2009, at 10:16 AM, Chris Lattner wrote:

>
> On May 1, 2009, at 10:13 AM, Dan Gohman wrote:
>
>> Author: djg
>> Date: Fri May  1 12:13:31 2009
>> New Revision: 70557
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=70557&view=rev
>> Log:
>> Actually insert inserted instructions into the InsertedValues map.
>
> Hi Dan,
>
> Should SCEV use AssertingVH's for the key of the Scalars map?  This
> would make sure that dangling pointer bugs don't happen.


This sounds like a good idea. I'll take a look at it when I'm
finished with a significant IndVarSimplify patch I'm working on.

Dan