rjmccall added a comment.

You only have one attempt to define the function here; I don't see the problem. 
 Recall that I said to add a flag to getOrCreateLLVMFunction that says whether 
the caller intends to define the function.  The rule should be that only 
callers that pass "true" should be allowed to assume that they'll get a normal 
llvm::Function back.  Everybody needs to be prepared to receive a bitcast.  
Whenever you find yourself needing to replace an existing function, just queue 
it up to be replaced at the end of IRGen.

I don't think we need to fall over ourselves ensuring that these "aliased" uses 
properly mark functions as used or instantiate templates or anything.


http://reviews.llvm.org/D11297



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to