[llvm-commits] [llvm] r68120 - in /llvm/trunk: lib/Transforms/Utils/InlineFunction.cpp test/Other/2009-03-31-CallGraph.ll

[Devang Patel]

On Mar 31, 2009, at 2:33 PM, Duncan Sands wrote:

> Hi Devang,
>
>> +            // We should be able to do this:
>> +            //   (*CG)[Caller]->replaceCallSite(CI, II);
>> +            // but that fails if the old call site isn't in the  
>> call graph,
>> +            // which, because of LLVM bug 3601, it sometimes isn't.
>
> wouldn't it be better to fix PR3601?

Eventually yes,  but I won't be able to get to it soon enough.
-
Devang

>
>
> Ciao,
>
> Duncan.
>
>> +            CallGraphNode *CGN = (*CG)[Caller];
>> +            for (CallGraphNode::iterator NI = CGN->begin(), NE =  
>> CGN->end();
>> +                 NI != NE; ++NI) {
>> +              if (NI->first == CI) {
>> +                NI->first = II;
>> +                break;
>> +              }
>> +            }
>> +          }
>> +