Fixed a profdata file size detection on Windows system.
authorVladimir Vereschaka <vvereschaka@accesssoftek.com>
Wed, 6 Nov 2019 01:09:50 +0000 (17:09 -0800)
committerVladimir Vereschaka <vvereschaka@accesssoftek.com>
Wed, 6 Nov 2019 01:09:50 +0000 (17:09 -0800)
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

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

index cc9b16a..9260279 100644 (file)
@@ -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.