[PATCH] D61103: [clang] Add tryToAttachCommentsToDecls method to ASTContext

Dmitri Gribenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 29 01:41:12 PDT 2019


gribozavr added a comment.

> Also, IIUC the test case that I deleted wasn't actually supposed to produce any diagnostics and the fact that it did was a bug. We could keep it as a regression test but I think it has a rather low value. WDYT?

What do you mean?  The issue that the test is trying to show is that a single //-line in a ///-comment breaks the doc comment and produces weird errors.  Instead it should tell the user that it looks like there's an unintended //-line.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61103/new/

https://reviews.llvm.org/D61103





More information about the cfe-commits mailing list