[PATCH] D17172: Minimum LTO implementation

[Rafael Espíndola via llvm-commits]

On 11 February 2016 at 18:42, Rui Ueyama <ruiu at google.com> wrote:
> ruiu added inline comments.
>
> ================
> Comment at: ELF/Driver.cpp:297-300
> @@ -292,1 +296,6 @@
>  template <class ELFT> void LinkerDriver::link(opt::InputArgList &Args) {
> +  // For LTO
> +  InitializeAllTargets();
> +  InitializeAllTargetMCs();
> +  InitializeAllAsmPrinters();
> +
> ----------------
> Do you know the cost of this initialization? (I mean if it is not cheap, say if it takes more than 10 milliseconds, then we want to do that lazily.)

about 130 microseconds on my machine with asserts enabled (according
to sys::TimeValue).

I will upload a patch with the other comments addressed.