ChuanqiXu added inline comments.

================
Comment at: clang/unittests/Format/FormatTest.cpp:7759-7761
+  // The default setting for PointerAlignment is PAS_Right.
+  // But if we set PointerAlignment as PAS_Left, the formatter
+  // would mis-format the pointer alignment.
----------------
HazardyKnusperkeks wrote:
> I don't think this should be in the tests, because it is not true (if your 
> patch works).
I'm confusing about your comment. What is not true? Do you mean if my patch 
works then the `co_return *a;` should be formatted as `co_return* a`?


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

https://reviews.llvm.org/D91245

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

Reply via email to