diff options
author | Kazu Hirata <kazu@google.com> | 2024-03-28 20:31:47 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-03-28 20:31:47 -0700 |
commit | c64a328cb4a32e81f8b694162750ec1b8823994c (patch) | |
tree | 58d359f565c3ec56202f893f5492bcc8e42175d6 | |
parent | 1d99d7a6f841de594b38936a4a73866b23a8b105 (diff) |
[ProfileData] Use size_t in PatchItem (NFC) (#87014)
size_t in PatchItem eliminates the need for casts.
-rw-r--r-- | llvm/lib/ProfileData/InstrProfWriter.cpp | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/llvm/lib/ProfileData/InstrProfWriter.cpp b/llvm/lib/ProfileData/InstrProfWriter.cpp index 8f067f8d05e2..1be23b852388 100644 --- a/llvm/lib/ProfileData/InstrProfWriter.cpp +++ b/llvm/lib/ProfileData/InstrProfWriter.cpp @@ -40,7 +40,7 @@ using namespace llvm; struct PatchItem { uint64_t Pos; // Where to patch. uint64_t *D; // Pointer to an array of source data. - int N; // Number of elements in \c D array. + size_t N; // Number of elements in \c D array. }; namespace llvm { @@ -69,7 +69,7 @@ public: const uint64_t LastPos = FDOStream.tell(); for (const auto &K : P) { FDOStream.seek(K.Pos); - for (int I = 0; I < K.N; I++) + for (size_t I = 0; I < K.N; I++) write(K.D[I]); } // Reset the stream to the last position after patching so that users @@ -80,7 +80,7 @@ public: raw_string_ostream &SOStream = static_cast<raw_string_ostream &>(OS); std::string &Data = SOStream.str(); // with flush for (const auto &K : P) { - for (int I = 0; I < K.N; I++) { + for (size_t I = 0; I < K.N; I++) { uint64_t Bytes = endian::byte_swap<uint64_t, llvm::endianness::little>(K.D[I]); Data.replace(K.Pos + I * sizeof(uint64_t), sizeof(uint64_t), @@ -707,9 +707,9 @@ Error InstrProfWriter::writeImpl(ProfOStream &OS) { {VTableNamesOffset, &VTableNamesSectionStart, 1}, // Patch the summary data. {SummaryOffset, reinterpret_cast<uint64_t *>(TheSummary.get()), - (int)(SummarySize / sizeof(uint64_t))}, + SummarySize / sizeof(uint64_t)}, {CSSummaryOffset, reinterpret_cast<uint64_t *>(TheCSSummary.get()), - (int)CSSummarySize}}; + CSSummarySize}}; OS.patch(PatchItems); } else { @@ -727,9 +727,9 @@ Error InstrProfWriter::writeImpl(ProfOStream &OS) { {TemporalProfTracesOffset, &TemporalProfTracesSectionStart, 1}, // Patch the summary data. {SummaryOffset, reinterpret_cast<uint64_t *>(TheSummary.get()), - (int)(SummarySize / sizeof(uint64_t))}, + SummarySize / sizeof(uint64_t)}, {CSSummaryOffset, reinterpret_cast<uint64_t *>(TheCSSummary.get()), - (int)CSSummarySize}}; + CSSummarySize}}; OS.patch(PatchItems); } |