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

Reply via email to