[llvm] bcbb121 - Fixed a profdata file size detection on Windows system.

Vladimir Vereschaka via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 5 17:40:06 PST 2019


Author: Vladimir Vereschaka
Date: 2019-11-05T17:09:50-08:00
New Revision: bcbb121ff6c8440382abfce8f2911a095f14602b

URL: https://github.com/llvm/llvm-project/commit/bcbb121ff6c8440382abfce8f2911a095f14602b
DIFF: https://github.com/llvm/llvm-project/commit/bcbb121ff6c8440382abfce8f2911a095f14602b.diff

LOG: Fixed a profdata file size detection on Windows system.

The space symbols are allowed in the group names on Windows system (as
example: Domain Users). In that case the test extracts a wrong field
from the output to get a size of the profdata file.

This patch avoids a printing of the group names in the test output and
extracts a proper field as a file size.

Differential Revision: https://reviews.llvm.org/D69317

Added: 
    

Modified: 
    llvm/test/tools/llvm-profdata/show-prof-size.test

Removed: 
    


################################################################################
diff  --git a/llvm/test/tools/llvm-profdata/show-prof-size.test b/llvm/test/tools/llvm-profdata/show-prof-size.test
index cc9b16aa08cc..926027992eee 100644
--- a/llvm/test/tools/llvm-profdata/show-prof-size.test
+++ b/llvm/test/tools/llvm-profdata/show-prof-size.test
@@ -1,5 +1,5 @@
 ; RUN: llvm-profdata merge -sample -extbinary -prof-sym-list=%S/Inputs/profile-symbol-list-1.text %S/Inputs/sample-profile.proftext -o %t.1.output
-; RUN: ls -l %t.1.output |tr -s ' ' |cut -f5 -d ' ' > %t.txt
+; RUN: ls -o %t.1.output |tr -s ' ' |cut -f4 -d ' ' > %t.txt
 ; RUN: llvm-profdata show -sample -show-sec-info-only %t.1.output >> %t.txt
 ; RUN: FileCheck %s --input-file=%t.txt
 ; Check llvm-profdata shows the correct file size.


        


More information about the llvm-commits mailing list