[PATCH] D81202: Fix the roundtrip test under llvm-profdata
Xun Li via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 4 16:37:01 PDT 2020
lxfind created this revision.
lxfind added reviewers: wmi, wenlei.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
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.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D81202
Files:
llvm/test/tools/llvm-profdata/roundtrip.test
Index: llvm/test/tools/llvm-profdata/roundtrip.test
===================================================================
--- llvm/test/tools/llvm-profdata/roundtrip.test
+++ llvm/test/tools/llvm-profdata/roundtrip.test
@@ -20,7 +20,7 @@
# 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D81202.268620.patch
Type: text/x-patch
Size: 920 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200604/36fd8d64/attachment.bin>
More information about the llvm-commits
mailing list