[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