[llvm] [llvm]Add a simple Telemetry framework (PR #102323)

James Henderson via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 17 01:40:39 PST 2024


================
@@ -187,31 +183,31 @@ To use Telemetry in your tool, you need to provide a concrete implementation of
 
   class MyDestination : public telemetry::Destination {
   public:
-    Error receiveEntry(const TelemetryInfo* Entry) override {
-      if (Error err = serializer.init()) {
-        return err;
+    Error receiveEntry(const TelemetryInfo *Entry) override {
+      if (Error Err = Serializer.init()) {
----------------
jh7370 wrote:

Nit: here and below LLVM style says we shouldn't have braces.

https://github.com/llvm/llvm-project/pull/102323


More information about the llvm-commits mailing list