feg208 added inline comments.
================ Comment at: clang/test/Format/struct-array-initializer.cpp:5 +// RUN: grep -Ev "// *[A-Z-]+:" %s \ +// RUN: | clang-format -style="{BasedOnStyle: LLVM, AlignArrayOfStructures: Left}" %s \ +// RUN: | FileCheck -strict-whitespace -check-prefix=CHECK2 %s ---------------- vitalybuka wrote: > This test is very flaky https://lab.llvm.org/buildbot/#/builders/109 > > ``` > ******************** TEST 'Clang :: Format/struct-array-initializer.cpp' > FAILED ******************** > Script: > -- > : 'RUN: at line 1'; grep -Ev "// *[A-Z-]+:" > /b/1/clang-x86_64-debian-fast/llvm.src/clang/test/Format/struct-array-initializer.cpp > | /b/1/clang-x86_64-debian-fast/llvm.obj/bin/clang-format > -style="{BasedOnStyle: LLVM, AlignArrayOfStructures: Right}" > /b/1/clang-x86_64-debian-fast/llvm.src/clang/test/Format/struct-array-initializer.cpp > | /b/1/clang-x86_64-debian-fast/llvm.obj/bin/FileCheck -strict-whitespace > -check-prefix=CHECK1 > /b/1/clang-x86_64-debian-fast/llvm.src/clang/test/Format/struct-array-initializer.cpp > : 'RUN: at line 4'; grep -Ev "// *[A-Z-]+:" > /b/1/clang-x86_64-debian-fast/llvm.src/clang/test/Format/struct-array-initializer.cpp > | /b/1/clang-x86_64-debian-fast/llvm.obj/bin/clang-format > -style="{BasedOnStyle: LLVM, AlignArrayOfStructures: Left}" > /b/1/clang-x86_64-debian-fast/llvm.src/clang/test/Format/struct-array-initializer.cpp > | /b/1/clang-x86_64-debian-fast/llvm.obj/bin/FileCheck -strict-whitespace > -check-prefix=CHECK2 > /b/1/clang-x86_64-debian-fast/llvm.src/clang/test/Format/struct-array-initializer.cpp > -- > Exit Code: 141 > Command Output (stderr): > -- > + : 'RUN: at line 1' > + /b/1/clang-x86_64-debian-fast/llvm.obj/bin/clang-format > '-style={BasedOnStyle: LLVM, AlignArrayOfStructures: Right}' > /b/1/clang-x86_64-debian-fast/llvm.src/clang/test/Format/struct-array-initializer.cpp > + grep -Ev '// *[A-Z-]+:' > /b/1/clang-x86_64-debian-fast/llvm.src/clang/test/Format/struct-array-initializer.cpp > + /b/1/clang-x86_64-debian-fast/llvm.obj/bin/FileCheck -strict-whitespace > -check-prefix=CHECK1 > /b/1/clang-x86_64-debian-fast/llvm.src/clang/test/Format/struct-array-initializer.cpp > -- > ******************** > ``` Yeah. I added you to a review to alter it to something I think will be less of an issue. Thanks by the by for getting the unused NetWidth variable Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101868/new/ https://reviews.llvm.org/D101868 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits