On Tue, Dec 20, 2016 at 5:58 PM, Kevin Puetz via Phabricator <revi...@reviews.llvm.org> wrote: > puetzk added a comment. > > I see that you added a FIXME mentioning that it should be C++-only, but I > don't see where you actually did anything that would make it so. What am I > missing?
You aren't missing anything, the fix isn't in yet -- the comment is calling out that we should fix this at some point in the future. > Also, did you mean to drop the changes to utils/TableGen/ClangAttrEmitter.cpp > and bring back the WarnDiag, "ExpectedEnumOrClass" @aaron.ballman asked for > in the first review? That change appears to have been lost, and I would like to see them brought back. I think they may have gotten lost during the rebase, as they were present in https://reviews.llvm.org/D26846?id=78569. ~Aaron > > > Repository: > rL LLVM > > https://reviews.llvm.org/D26846 > > > _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
