[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