[PATCH] D25384: [ThinLTO] Record references to aliases to calls

[Teresa Johnson via llvm-commits]

tejohnson added inline comments.


================
Comment at: lib/Analysis/ModuleSummaryAnalysis.cpp:58
         // specially by WriteFunction and are added to a separate list.
-        if (!(CS && CS.isCallee(&OI)))
+        if (!(CS && CS.isCallee(&OI) && CS.getCalledFunction()))
           RefEdges.insert(Operand);
----------------
mehdi_amini wrote:
> I don't understand this change. I seem to me that you're adding a reference when we're calling an alias. Why isn't the alias added as a callee?
Because there is no called function (it's null). Note we can still find the aliasee function in the ref graph via the alias summary. Maybe we should add a call edge to the alias regardless, so we could import via the alias summary...let me try that . 


https://reviews.llvm.org/D25384