aaron.ballman added inline comments.
================
Comment at: clang/docs/ClangPlugins.rst:74
+
+ class ExampleAttrInfo : public ParsedAttrInfo {
+ public:
----------------
We should be documenting the fields of `ParsedAttrInfo` that the user would be
interacting with. We should also consider whether we are introducing a new
stability guarantee with that interface and document accordingly. I think we
should not promise that this structure will be stable from release to release,
but it will be stable within a given release (e.g., the type can change between
Clang 11 -> Clang 12, but the type will not change between Clang 11 -> Clang
11.0.1).
================
Comment at: clang/lib/Sema/SemaDeclAttr.cpp:6701-6702
+ if (AL.getInfo().handleDeclAttribute(S, D, AL))
+ return;
+
----------------
I think this might make more logical sense as part of the `default` label in
the `switch` statement.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D31342/new/
https://reviews.llvm.org/D31342
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
