Fix the roundtrip test under llvm-profdata
authorXun Li <xun@fb.com>
Fri, 5 Jun 2020 15:51:54 +0000 (08:51 -0700)
committerWenlei He <aktoon@gmail.com>
Fri, 5 Jun 2020 15:52:21 +0000 (08:52 -0700)
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

llvm/test/tools/llvm-profdata/roundtrip.test

index ce20d63..a458b21 100644 (file)
@@ -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