[lld] r189310 - Fix use of temporary strings.

Shankar Easwaran shankare at codeaurora.org
Mon Aug 26 21:57:10 PDT 2013


How are these temporary strings ? They all are pointing to the arguments 
argv[].

Is this to handle the .directive file ?

Thanks

Shankar Easwaran

On 8/26/2013 11:27 PM, Rui Ueyama wrote:
> Author: ruiu
> Date: Mon Aug 26 23:27:18 2013
> New Revision: 189310
>
> URL: http://llvm.org/viewvc/llvm-project?rev=189310&view=rev
> Log:
> Fix use of temporary strings.
>
> Modified:
>      lld/trunk/lib/Driver/WinLinkDriver.cpp
>
> Modified: lld/trunk/lib/Driver/WinLinkDriver.cpp
> URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/Driver/WinLinkDriver.cpp?rev=189310&r1=189309&r2=189310&view=diff
> ==============================================================================
> --- lld/trunk/lib/Driver/WinLinkDriver.cpp (original)
> +++ lld/trunk/lib/Driver/WinLinkDriver.cpp Mon Aug 26 23:27:18 2013
> @@ -350,12 +350,12 @@ bool WinLinkDriver::parse(int argc, cons
>   
>       case OPT_entry:
>         // handle /entry
> -      ctx.setEntrySymbolName(inputArg->getValue());
> +      ctx.setEntrySymbolName(ctx.allocateString(inputArg->getValue()));
>         break;
>   
>       case OPT_libpath:
>         // handle /libpath
> -      ctx.appendInputSearchPath(inputArg->getValue());
> +      ctx.appendInputSearchPath(ctx.allocateString(inputArg->getValue()));
>         break;
>   
>       case OPT_force:
> @@ -408,12 +408,12 @@ bool WinLinkDriver::parse(int argc, cons
>   
>       case OPT_incl:
>         // handle /incl
> -      ctx.addInitialUndefinedSymbol(inputArg->getValue());
> +      ctx.addInitialUndefinedSymbol(ctx.allocateString(inputArg->getValue()));
>         break;
>   
>       case OPT_out:
>         // handle /out
> -      ctx.setOutputPath(inputArg->getValue());
> +      ctx.setOutputPath(ctx.allocateString(inputArg->getValue()));
>         break;
>   
>       case OPT_INPUT: {
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>


-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation




More information about the llvm-commits mailing list