[llvm] [memprof] Drop curly braces on small for loops (NFC) (PR #119516)
via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 10 23:07:51 PST 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-pgo
Author: Kazu Hirata (kazutakahirata)
<details>
<summary>Changes</summary>
---
Full diff: https://github.com/llvm/llvm-project/pull/119516.diff
1 Files Affected:
- (modified) llvm/unittests/ProfileData/MemProfTest.cpp (+4-8)
``````````diff
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));
``````````
</details>
https://github.com/llvm/llvm-project/pull/119516
More information about the llvm-commits
mailing list