[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