[PATCH] D60663: Time profiler: small fixes and optimizations

Anton Afanasyev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 15 12:06:25 PDT 2019


anton-afanasyev added a comment.

In D60663#1465721 <https://reviews.llvm.org/D60663#1465721>, @lebedev.ri wrote:

> Looks good ignoring the json bits.
>
> Re license:
>
> > https://reviews.llvm.org/D58675
> >  This is the first part of time tracing system, I have splitted them cause this part is mostly written by Aras Pranckevicius except of several minor fixes concerning formatting.
>
> So i can't and won't claim any legal knowledge, but it maybe would be good for him to at least comment here, that he is ok with this?


I've communicated to Aras by mail, he is ok with new license header:

> "Sure! I started my branch before the LLVM license change. The new one is fine."


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60663/new/

https://reviews.llvm.org/D60663





More information about the llvm-commits mailing list