AaronBallman wrote: > I don't think `setInvalidDecl` is needed in this case. The attribute is > rejected before it is added to the declaration and the declaration itself > remains valid.
Oh, I see the logic you're using -- even though the declaration was invalid enough to be diagnosed, we didn't add the attribute to the AST node and so the declaration is sufficiently valid again. That's subtle but reasonable IMO. https://github.com/llvm/llvm-project/pull/194856 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
