Fixed a profdata file size detection on Windows system.

[vvereschaka]

Summary

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.

[andreil99]

Thanks, Vlad!

[wmi]

Sorry I missed the patch. Thanks for fixing the test!

[mgorny]

This broke the NetBSD buildbot: http://lab.llvm.org:8014/builders/netbsd-amd64/builds/140/steps/run%20unit%20tests/logs/FAIL%3A%20LLVM%3A%3Ashow-prof-size.test

The -o option is not portable, and does something completely different on NetBSD:

-o      Include the file flags in a long (-l) output.  If no file flags
        are set, “-” is displayed.  (See chflags(1) for a list of
        possible flags and their meanings.)

The same is true for at least FreeBSD and macOS. So this patch breaks more platforms than it fixes.

[krytarowski]

-o is an unportable extension.

https://pubs.opengroup.org/onlinepubs/9699919799/utilities/ls.html

[vvereschaka]

Thanks @mgorny and @krytarowski for letting me know.

Looks like there is no way to make a portable fix using 'ls' tool.
I am going to revert this commit.

[vvereschaka]

The commit has been reverted by 3d3445e3e691a34a72643d07e675c618adeddafc

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

 ; 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.