[PATCH] D69317: Fixed a profdata file size detection on Windows system.
Michał Górny via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Nov 10 07:06:38 PST 2019
mgorny added a comment.
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.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69317/new/
https://reviews.llvm.org/D69317
More information about the llvm-commits
mailing list