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

[Duncan Sands]

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?

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;
> +              }
> +            }
> +          }
> +