[cfe-dev] TargetInfo::CreateTargetInfo() takes owneship by reference

Mehdi AMINI mehdi.amini at silkan.com
Thu Nov 15 20:14:39 PST 2012


Le 09/11/2012 17:04, Mehdi AMINI a écrit :
> Hi,
> r165943 changed *TargetInfo::CreateTargetInfo() so that it takes
> ownership of TargetOptions &Opts which is passed by reference.
> While it seems to me that it is not a good idea to take ownership from a
> reference in general (any guidelines about that in LLVM/Clang?), I feel
> it should be at least clearly stated in the comment before the function.

Does anyone care ? Client code like the one presented here: 
result in double-free corruption.


More information about the cfe-dev mailing list