On Wed, Dec 21, 2016 at 8:36 AM, Aaron Ballman <aaron.ball...@gmail.com>
wrote:

> 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.
>

I removed the tablegen logic because it was dead after changing the uuid
SubjectList to this:
  let Subjects = SubjectList<[Record, Enum], WarnDiag,
"ExpectedEnumOrClass">;

It was necessary to go from CXXRecord to Record here so that we would
accept __uuid on structs in C and then reject the attribute as being
C++-only later.

Feel free to add the case back. From my reading of the code, it seems
like CalculateDiagnostic isn't intended to cover every case.
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to