[llvm] [llvm]Add a simple Telemetry framework (PR #102323)
Vy Nguyen via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 23 14:42:43 PDT 2024
oontvoo wrote:
> I've gone through the implementation, but run out of time to look at the tests today. I'll look at those more next week, if I can. If there are any elements I've missed replying to in the non-test parts of this PR, please call them out, as I've left this too long to be hot on the whole conversation.
>
> I've spent over 2 hours looking at this today, writing up thoughts and comments and generally thinking about the design and
Thanks! I've responded to the 3 comments, which I think are the major ones .( I'll address your other comments later since they seem quite straight forward).
https://github.com/llvm/llvm-project/pull/102323
More information about the llvm-commits
mailing list