[PATCH] D146759: Declaration fragments for properties include additional property attributes that were not specified in the sources proposed solution
Zixu Wang via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 27 14:29:50 PDT 2023
zixuw requested changes to this revision.
zixuw added a comment.
This revision now requires changes to proceed.
Please update/add test cases.
================
Comment at: CONTRIBUTING.md:10-18
+
+## Getting in touch
+
+Join the [LLVM Discourse forums](https://discourse.llvm.org/), [Discord
+chat](https://discord.gg/xS7Z362), or #llvm IRC channel on
+[OFTC](https://oftc.net/).
+
----------------
Do not include irrelevant changes.
================
Comment at: clang/lib/ExtractAPI/DeclarationFragments.cpp:629-630
- const auto Attributes = Property->getPropertyAttributes();
+ // changed the getPropertyAttributes() to getPropertyAttributesAsWritten()
+ // in following line when calling Property->getPropertyAttributesAsWritten();
+ const auto Attributes = Property->getPropertyAttributesAsWritten();
----------------
Do not use comments to describe the change you've made.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146759/new/
https://reviews.llvm.org/D146759
More information about the llvm-commits
mailing list