[PATCH] D56924: Special case ObjCPropertyDecl for printing

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 18 08:10:52 PDT 2019


dexonsmith added subscribers: jkorous, benlangmuir.
dexonsmith added inline comments.


================
Comment at: unittests/AST/NamedDeclPrinterTest.cpp:220
+    "property",
+    "Obj::property"));
+}
----------------
gribozavr wrote:
> I don't think that `Obj::property` is the preferred syntax. `Obj.property`? I'd want a review from someone from Apple to confirm.
@arphaman, @jkorous, or @benlangmuir: can you check on this?


Repository:
  rC Clang

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

https://reviews.llvm.org/D56924





More information about the cfe-commits mailing list