[llvm] 45f58bf - Fix the roundtrip test under llvm-profdata
Wenlei He via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 5 08:52:54 PDT 2020
Author: Xun Li
Date: 2020-06-05T08:52:21-07:00
New Revision: 45f58bf469a77244d064ce13f9a81ecd16dc462a
URL: https://github.com/llvm/llvm-project/commit/45f58bf469a77244d064ce13f9a81ecd16dc462a
DIFF: https://github.com/llvm/llvm-project/commit/45f58bf469a77244d064ce13f9a81ecd16dc462a.diff
LOG: Fix the roundtrip test under llvm-profdata
Summary:
According to the comments, we want to convert the profile into two binary formats, and then into the md5text format.
We seems to have ignored the intermediate files.
This patch uses them to complete the full roundtrips.
Reviewers: wmi, wenlei
Reviewed By: wmi
Subscribers: llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D81202
Added:
Modified:
llvm/test/tools/llvm-profdata/roundtrip.test
Removed:
################################################################################
diff --git a/llvm/test/tools/llvm-profdata/roundtrip.test b/llvm/test/tools/llvm-profdata/roundtrip.test
index ce20d633a74a..a458b21ac083 100644
--- a/llvm/test/tools/llvm-profdata/roundtrip.test
+++ b/llvm/test/tools/llvm-profdata/roundtrip.test
@@ -20,7 +20,7 @@ RUN:
diff -b %t.4.proftext %S/Inputs/sample-profile.proftext
# Trip from text --> compbinary --> md5text
# Compare the two md5 texts
RUN: llvm-profdata merge --sample --compbinary -output=%t.6.profdata %S/Inputs/sample-profile.proftext
-RUN: llvm-profdata merge --sample --text -output=%t.6.proftext %S/Inputs/sample-profile.proftext
+RUN: llvm-profdata merge --sample --text -output=%t.6.proftext %t.6.profdata
RUN: llvm-profdata merge --sample --extbinary -use-md5 -output=%t.7.profdata %S/Inputs/sample-profile.proftext
-RUN: llvm-profdata merge --sample --text -output=%t.7.proftext %S/Inputs/sample-profile.proftext
+RUN: llvm-profdata merge --sample --text -output=%t.7.proftext %t.7.profdata
RUN:
diff -b %t.6.proftext %t.7.proftext
More information about the llvm-commits
mailing list