[PATCH] D24492: [LTO] Switch to the new resolution-based API.

Rafael EspĂ­ndola via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 28 11:45:18 PDT 2016


please base on top of trunk.

On 28 September 2016 at 11:34, Davide Italiano <dccitaliano at gmail.com> wrote:
> davide added inline comments.
>
> ================
> Comment at: lld/ELF/LTO.cpp:154
> @@ +153,3 @@
> +  for (unsigned I = 0; I != MaxTasks; ++I) {
> +    if (!Buff[I].empty()) {
> +      // Do we want to move this to the LTO API?
> ----------------
> hmm, I still need `I` to be passed to saveBuffer, no? Do you want me to use a separate variable incremented at each iteration?
>
>
> https://reviews.llvm.org/D24492
>
>
>


More information about the llvm-commits mailing list