[llvm] [MemProf] Include <ctime> to avoid MSVC failure (PR #114246)
Teresa Johnson via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 30 08:04:15 PDT 2024
https://github.com/teresajohnson created https://github.com/llvm/llvm-project/pull/114246
My change in bb3915149a7c9b1660db9caebfc96343352e8454 added a call to
std::time which worked generally as there must be some transitive
include of <ctime>. However, I saw one MSVC bot failure:
InstrProfWriter.cpp(202): error C2039: 'time': is not a member of 'std'
from https://lab.llvm.org/buildbot/#/builders/63/builds/2325.
Presumably explictly including <ctime> should fix this.
>From 3656020ab430700be2e8557f5d9921f39bcfb2e4 Mon Sep 17 00:00:00 2001
From: Teresa Johnson <tejohnson at google.com>
Date: Wed, 30 Oct 2024 08:01:03 -0700
Subject: [PATCH] [MemProf] Include <ctime> to avoid MSVC failure
My change in bb3915149a7c9b1660db9caebfc96343352e8454 added a call to
std::time which worked generally as there must be some transitive
include of <ctime>. However, I saw one MSVC bot failure:
InstrProfWriter.cpp(202): error C2039: 'time': is not a member of 'std'
from https://lab.llvm.org/buildbot/#/builders/63/builds/2325.
Presumably explictly including <ctime> should fix this.
---
llvm/lib/ProfileData/InstrProfWriter.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/llvm/lib/ProfileData/InstrProfWriter.cpp b/llvm/lib/ProfileData/InstrProfWriter.cpp
index f09241681b92a5..0ab9f942a08589 100644
--- a/llvm/lib/ProfileData/InstrProfWriter.cpp
+++ b/llvm/lib/ProfileData/InstrProfWriter.cpp
@@ -29,6 +29,7 @@
#include "llvm/Support/OnDiskHashTable.h"
#include "llvm/Support/raw_ostream.h"
#include <cstdint>
+#include <ctime>
#include <memory>
#include <string>
#include <tuple>
More information about the llvm-commits
mailing list