[PATCH] D139603: [llvm-profdata] Add option to cap profile output size

Douglas Yung via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 9 20:00:31 PST 2023


dyung added inline comments.


================
Comment at: llvm/test/tools/llvm-profdata/output-size-limit.test:61
+RUN: llvm-profdata merge --sample --extbinary --output-size-limit=489  %p/Inputs/sample-profile.proftext -o %t.output
+RUN: test $(stat -c %%s %t.output) -le 489
+RUN: llvm-profdata merge --sample --text %t.output | FileCheck %s --check-prefix=TEST_EXTBINARY1
----------------
On Windows this seems to be expanded in a way you probably did not expect:
(https://lab.llvm.org/buildbot/#/builders/216/builds/15553/steps/7/logs/FAIL__LLVM__output-size-limit_test)
`"$(stat" "-c" "%s" "Z:\test\build\test\tools\llvm-profdata\Output\output-size-limit.test.tmp.output)"`


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139603/new/

https://reviews.llvm.org/D139603



More information about the llvm-commits mailing list