From bcbb121ff6c8440382abfce8f2911a095f14602b Mon Sep 17 00:00:00 2001 From: Vladimir Vereschaka Date: Tue, 5 Nov 2019 17:09:50 -0800 Subject: [PATCH] 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 --- llvm/test/tools/llvm-profdata/show-prof-size.test | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/llvm/test/tools/llvm-profdata/show-prof-size.test b/llvm/test/tools/llvm-profdata/show-prof-size.test index cc9b16a..9260279 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. -- 2.7.4