[llvm] [memprof] Drop curly braces on small for loops (NFC) (PR #119516)
Kazu Hirata via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 10 23:07:17 PST 2024
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/119516
None
>From ce9619b7d5042883cd11cf4311f7e361803d9169 Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Sat, 7 Dec 2024 11:34:06 -0800
Subject: [PATCH] [memprof] Drop curly braces on small for loops (NFC)
---
llvm/unittests/ProfileData/MemProfTest.cpp | 12 ++++--------
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/llvm/unittests/ProfileData/MemProfTest.cpp b/llvm/unittests/ProfileData/MemProfTest.cpp
index 5886d3e2bcf842..3d975f96ed96cd 100644
--- a/llvm/unittests/ProfileData/MemProfTest.cpp
+++ b/llvm/unittests/ProfileData/MemProfTest.cpp
@@ -161,9 +161,8 @@ TEST(MemProf, FillsValue) {
/*KeepName=*/true);
llvm::DenseMap<llvm::GlobalValue::GUID, MemProfRecord> Records;
- for (const auto &Pair : Reader) {
+ for (const auto &Pair : Reader)
Records.insert({Pair.first, Pair.second});
- }
// Mock program pseudocode and expected memprof record contents.
//
@@ -396,9 +395,8 @@ TEST(MemProf, SymbolizationFilter) {
RawMemProfReader Reader(std::move(Symbolizer), Seg, Prof, CSM);
llvm::SmallVector<MemProfRecord, 1> Records;
- for (const auto &KeyRecordPair : Reader) {
+ for (const auto &KeyRecordPair : Reader)
Records.push_back(KeyRecordPair.second);
- }
ASSERT_THAT(Records, SizeIs(1));
ASSERT_THAT(Records[0].AllocSites, SizeIs(1));
@@ -428,9 +426,8 @@ TEST(MemProf, BaseMemProfReader) {
MemProfReader Reader(std::move(MemProfData));
llvm::SmallVector<MemProfRecord, 1> Records;
- for (const auto &KeyRecordPair : Reader) {
+ for (const auto &KeyRecordPair : Reader)
Records.push_back(KeyRecordPair.second);
- }
ASSERT_THAT(Records, SizeIs(1));
ASSERT_THAT(Records[0].AllocSites, SizeIs(1));
@@ -463,9 +460,8 @@ TEST(MemProf, BaseMemProfReaderWithCSIdMap) {
MemProfReader Reader(std::move(MemProfData));
llvm::SmallVector<MemProfRecord, 1> Records;
- for (const auto &KeyRecordPair : Reader) {
+ for (const auto &KeyRecordPair : Reader)
Records.push_back(KeyRecordPair.second);
- }
ASSERT_THAT(Records, SizeIs(1));
ASSERT_THAT(Records[0].AllocSites, SizeIs(1));
More information about the llvm-commits
mailing list