[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