sbenza added a comment. In http://reviews.llvm.org/D21036#450935, @aaron.ballman wrote:
> In http://reviews.llvm.org/D21036#450106, @sbenza wrote: > > > I think this would be more interesting with macros. > > Eg triggering in code like this: > > > > #define FOO(type, op) const type& X = op() > > FOO(int*, bar); > > > Sorry, I originally missed this comment. > > That might be a reasonable addition to this check, but strikes me as a future > enhancement. Are you looking for the patch to cover this case initially, or > is it fine for a follow-up? Yes, a follow up is ok. I was just making sure this is considered. http://reviews.llvm.org/D21036 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
