[llvm] r365450 - [llvm-profdata] Don't make the output overwrite the input file.

Haojian Wu via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 9 02:57:45 PDT 2019


Author: hokein
Date: Tue Jul  9 02:57:45 2019
New Revision: 365450

URL: http://llvm.org/viewvc/llvm-project?rev=365450&view=rev
Log:
[llvm-profdata] Don't make the output overwrite the input file.

Some file systems may not allow this behavior, the test fails on our internal
system ("Permission denied").

Modified:
    llvm/trunk/test/tools/llvm-profdata/same-filename.test

Modified: llvm/trunk/test/tools/llvm-profdata/same-filename.test
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/llvm-profdata/same-filename.test?rev=365450&r1=365449&r2=365450&view=diff
==============================================================================
--- llvm/trunk/test/tools/llvm-profdata/same-filename.test (original)
+++ llvm/trunk/test/tools/llvm-profdata/same-filename.test Tue Jul  9 02:57:45 2019
@@ -2,8 +2,8 @@ Test the input file names overlap with t
 
 This is OK for merging instrument profiles.
 RUN: cp %S/Inputs/same-name-1.proftext %t.0.proftext
-RUN: llvm-profdata merge -o %t.0.proftext -text %t.0.proftext %t.0.proftext
-RUN: llvm-profdata show -counts -all-functions %t.0.proftext -o %t_show_0
+RUN: llvm-profdata merge -o %t.0.profdata -text %t.0.proftext %t.0.proftext
+RUN: llvm-profdata show -counts -all-functions %t.0.profdata -o %t_show_0
 RUN: llvm-profdata show -counts -all-functions %S/Inputs/same-name-2.proftext -o %t_show_1
 RUN: diff %t_show_0 %t_show_1
 RUN: llvm-profdata merge -o %t.1.profdata %S/Inputs/same-name-1.proftext
@@ -17,8 +17,8 @@ CHECK: llvm-profdata{{.*}} show: Input f
 
 This is OK for merging sample fdo profiles.
 RUN: cp %S/Inputs/same-name-3.proftext %t.3.proftext
-RUN: llvm-profdata merge --sample -o %t.3.proftext -text %t.3.proftext %t.3.proftext
-RUN: llvm-profdata show --sample -counts -all-functions %t.3.proftext -o %t_show_3
+RUN: llvm-profdata merge --sample -o %t.3.profdata -text %t.3.proftext %t.3.proftext
+RUN: llvm-profdata show --sample -counts -all-functions %t.3.profdata -o %t_show_3
 RUN: llvm-profdata show --sample -counts -all-functions %S/Inputs/same-name-4.proftext -o %t_show_4
 RUN: diff %t_show_3 %t_show_4
 RUN: llvm-profdata merge --sample -o %t.5.profdata %S/Inputs/same-name-3.proftext




More information about the llvm-commits mailing list