[PATCH] D56555: Add Attribute to define nonlazy objc classes
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 11 05:41:49 PST 2019
aaron.ballman requested changes to this revision.
aaron.ballman added a comment.
This revision now requires changes to proceed.
The patch is missing SemaObjC tests that ensure the attribute only appertains to the expected subjects, accepts no args, etc.
================
Comment at: clang/include/clang/Basic/Attr.td:1696
+ let Subjects = SubjectList<[ObjCInterface], ErrorDiag>;
+ let Documentation = [Undocumented];
+}
----------------
No new undocumented attributes, please.
Does this attribute make sense when compiling a non-ObjC compiland? e.g., should this also have a `let LangOpts = [ObjC];` as well?
================
Comment at: clang/lib/Sema/SemaDeclAttr.cpp:6377-6379
+ case ParsedAttr::AT_ObjCNonLazyClass:
+ handleSimpleAttribute<ObjCNonLazyClassAttr>(S, D, AL);
+ break;
----------------
Formatting is incorrect here -- you should run the patch through clang-format.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56555/new/
https://reviews.llvm.org/D56555
More information about the cfe-commits
mailing list