diff options
author | Amir Ayupov <aaupov@fb.com> | 2024-01-18 19:59:09 -0800 |
---|---|---|
committer | Amir Ayupov <aaupov@fb.com> | 2024-01-18 19:59:09 -0800 |
commit | 9fec33aadc56c8c4ad3778a92dc0aaa3201a63ae (patch) | |
tree | 4f3bea9b1b20a36bb0ee374a6fca3b0987ec00c8 /bolt | |
parent | 82bc33ea3f1a539be50ed46919dc53fc6b685da9 (diff) | |
download | llvm-9fec33aadc56c8c4ad3778a92dc0aaa3201a63ae.zip llvm-9fec33aadc56c8c4ad3778a92dc0aaa3201a63ae.tar.gz llvm-9fec33aadc56c8c4ad3778a92dc0aaa3201a63ae.tar.bz2 |
Revert "[BOLT] Fix unconditional output of boltedcollection in merge-fdata (#78653)"
This reverts commit 82bc33ea3f1a539be50ed46919dc53fc6b685da9.
Accidentally pushed unrelated changes.
Diffstat (limited to 'bolt')
-rw-r--r-- | bolt/lib/Core/BinaryEmitter.cpp | 1 | ||||
-rw-r--r-- | bolt/test/X86/merge-fdata-bat-mode.test | 3 | ||||
-rw-r--r-- | bolt/test/X86/merge-fdata-nobat-mode.test | 6 | ||||
-rw-r--r-- | bolt/tools/merge-fdata/merge-fdata.cpp | 2 |
4 files changed, 3 insertions, 9 deletions
diff --git a/bolt/lib/Core/BinaryEmitter.cpp b/bolt/lib/Core/BinaryEmitter.cpp index 190c700..3bff312 100644 --- a/bolt/lib/Core/BinaryEmitter.cpp +++ b/bolt/lib/Core/BinaryEmitter.cpp @@ -485,7 +485,6 @@ void BinaryEmitter::emitFunctionBody(BinaryFunction &BF, FunctionFragment &FF, // This assumes the second instruction in the macro-op pair will get // assigned to its own MCRelaxableFragment. Since all JCC instructions // are relaxable, we should be safe. - Streamer.emitNeverAlignCodeAtEnd(/*Alignment to avoid=*/64, *BC.STI); } if (!EmitCodeOnly) { diff --git a/bolt/test/X86/merge-fdata-bat-mode.test b/bolt/test/X86/merge-fdata-bat-mode.test index 41738e1..ea3e4c3 100644 --- a/bolt/test/X86/merge-fdata-bat-mode.test +++ b/bolt/test/X86/merge-fdata-bat-mode.test @@ -4,5 +4,6 @@ RUN: merge-fdata %S/Inputs/bat_profile_1.fdata \ RUN: %S/Inputs/bat_profile_2.fdata \ RUN: | FileCheck %s --check-prefix=CHECK-FDATA -CHECK-FDATA: boltedcollection CHECK-FDATA: 1 main 451 1 SolveCubic 0 0 302 + + diff --git a/bolt/test/X86/merge-fdata-nobat-mode.test b/bolt/test/X86/merge-fdata-nobat-mode.test deleted file mode 100644 index 870d9f8..0000000 --- a/bolt/test/X86/merge-fdata-nobat-mode.test +++ /dev/null @@ -1,6 +0,0 @@ -# Check that merge-fdata tool doesn't spuriously print boltedcollection - -RUN: merge-fdata %S/Inputs/blarge.fdata %S/Inputs/blarge.fdata \ -RUN: | FileCheck %s --check-prefix=CHECK-FDATA - -CHECK-FDATA-NOT: boltedcollection diff --git a/bolt/tools/merge-fdata/merge-fdata.cpp b/bolt/tools/merge-fdata/merge-fdata.cpp index 104991d..e21aeba 100644 --- a/bolt/tools/merge-fdata/merge-fdata.cpp +++ b/bolt/tools/merge-fdata/merge-fdata.cpp @@ -329,7 +329,7 @@ void mergeLegacyProfiles(const SmallVectorImpl<std::string> &Filenames) { MergedProfile.insert_or_assign(Key, Count); } - if (BoltedCollection.value_or(false)) + if (BoltedCollection) output() << "boltedcollection\n"; for (const auto &[Key, Value] : MergedProfile) output() << Key << " " << Value << "\n"; |