aaron.ballman added a comment.
In http://reviews.llvm.org/D16708#340359, @gottesmm wrote:
> I think that my response via email did not hit phabriactor. So sorry for the
> delay.
No worries!
> Yes there is a forthcoming patch for CodeGen which will place an attribute on
> the relevant functio
gottesmm added inline comments.
Comment at: include/clang/Basic/Attr.td:540
@@ +539,3 @@
+ let Subjects = SubjectList<[Function]>;
+ let Documentation = [Undocumented];
+}
aaron.ballman wrote:
> Please, no undocumented new attributes. You should modify AttrDocs.
gottesmm added a comment.
I think that my response via email did not hit phabriactor. So sorry for the
delay.
Yes there is a forthcoming patch for CodeGen which will place an attribute on
the relevant functions. The attribute will be queried in the middle end
optimizer. The reason why there ha
> On Jan 29, 2016, at 6:02 AM, Aaron Ballman wrote:
>
> aaron.ballman added a subscriber: aaron.ballman.
> aaron.ballman added a reviewer: aaron.ballman.
> aaron.ballman added a comment.
>
> It's a bit strange to add an attribute that has absolutely no semantic effect
> whatsoever. Where is th
aaron.ballman added a subscriber: aaron.ballman.
aaron.ballman added a reviewer: aaron.ballman.
aaron.ballman added a comment.
It's a bit strange to add an attribute that has absolutely no semantic effect
whatsoever. Where is this attribute intended to be queried within the compiler?
Are there a
gottesmm updated this revision to Diff 46354.
gottesmm added a comment.
Added documentation.
http://reviews.llvm.org/D16708
Files:
include/clang/Basic/Attr.td
lib/Sema/SemaDeclAttr.cpp
test/SemaObjC/attr-cf_no_release.m
Index: test/SemaObjC/attr-cf_no_release.m
==
