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

Anton Afanasyev via Phabricator via cfe-commits cfe-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."

  rG LLVM Github Monorepo



More information about the cfe-commits mailing list