owenpan added a comment.

In D127484#3574992 <https://reviews.llvm.org/D127484#3574992>, @MyDeveloperDay 
wrote:

> Oh gosh! It hard to make such changes and someone else not call them a 
> regression, I’m not sure how I feel. We do kind of have to be able to fix 
> bugs and if this gets closer to gnu style then I guess it’s better

+1.



================
Comment at: clang/unittests/Format/FormatTest.cpp:23954
+               "\t{\n"
+               "\t  printf (\"%d, %d\\n\", i, j);\n"
+               "\t}\n"
----------------
To simplify the test case a little bit.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127484

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to