[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