[PATCH] D69317: Fixed a profdata file size detection on Windows system.

[Vlad Vereschaka via Phabricator via llvm-commits]

vvereschaka created this revision.
vvereschaka added a reviewer: wmi.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

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.


Repository:
  rL LLVM

https://reviews.llvm.org/D69317

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


Index: llvm/test/tools/llvm-profdata/show-prof-size.test
===================================================================
--- llvm/test/tools/llvm-profdata/show-prof-size.test
+++ 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.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D69317.226066.patch
Type: text/x-patch
Size: 691 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191022/ee15e1de/attachment.bin>