[PATCH] D55168: [test] Fix use of 'sort -b' in SimpleLoopUnswitch on NetBSD
Fedor Sergeev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Dec 2 07:46:17 PST 2018
fedor.sergeev added a comment.
In D55168#1315857 <https://reviews.llvm.org/D55168#1315857>, @mgorny wrote:
> i.e. the problem is that 'depth 2' messages are indented, and (for some reason) the tests want them all sorted by message.
The tests are trying to count loops at different depths, using -COUNT-<NUM> directives, e.g. 11 loops at depth 1, 40 loops at depth 2.
Alternative to sorting would be to implement support for -COUNT-<NUM>-DAG.
-b was added to sort to fix buildbot failures.
I could only speculate that under certain locales whitespaces might lead to different ordering.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55168/new/
https://reviews.llvm.org/D55168
More information about the llvm-commits
mailing list