owenpan added a comment.

Should we revert this patch?



================
Comment at: clang/unittests/Format/FormatTest.cpp:5175
   verifyFormat("#pragma omp threadprivate( \\\n"
-               "    y)), // expected-warning",
+               "        y)), // expected-warning",
                getLLVMStyleWithColumns(28));
----------------
Why was this test case changed? It seemed to be related to the regression 
mentioned in D136100#3988574.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136100

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D136100: [clang-forma... Owen Pan via Phabricator via cfe-commits

Reply via email to