================ @@ -104,6 +105,23 @@ struct llvm::TimeTraceProfilerEntry { } }; +struct InProgressEntry { + std::unique_ptr<TimeTraceProfilerEntry> Event; ---------------- ilya-biryukov wrote:
Could we store it directly as `TimeTraceProfilerEntry` without unique_ptr? We are already creating `InProgressEntry` in the heap, so the stability of pointers for its members should be guaranteed. Having exactly one of `InProgressEntry` or the corresponding `TimeTraceProfilerEntry` in the heap should be enough. https://github.com/llvm/llvm-project/pull/103039 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits