t.p.northover closed this revision. t.p.northover marked an inline comment as done. t.p.northover added a comment.
Thanks, committed as e94fada045fe <https://reviews.llvm.org/rGe94fada045fe88787a414e3307412c1cc3a1b259> with the extra tests. ================ Comment at: clang/include/clang/Basic/AttrDocs.td:4404 + +A context parameter must have pointer or reference type. + }]; ---------------- aaron.ballman wrote: > Should we specify whether pointer to members or ObjC object pointers are fine > (given that those are both somewhat "odd" pointer types)? Function pointers? > > (We may want to clarify this in the other docs in a follow-up if we think > this is unclear.) I don't think either of those are banned, but it's kind of a "play silly games win silly prizes" situation. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95228/new/ https://reviews.llvm.org/D95228 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
