r339387 - Revert -r339382, which apparently breaks the Windows build.

Richard Smith via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 9 14:15:52 PDT 2018


On Thu, 9 Aug 2018 at 14:14, Erich Keane via cfe-commits <
cfe-commits at lists.llvm.org> wrote:

> Author: erichkeane
> Date: Thu Aug  9 14:13:46 2018
> New Revision: 339387
>
> URL: http://llvm.org/viewvc/llvm-project?rev=339387&view=rev
> Log:
> Revert -r339382, which apparently breaks the Windows build.
>
> Modified:
>     cfe/trunk/include/clang/Sema/ParsedAttr.h
>
> Modified: cfe/trunk/include/clang/Sema/ParsedAttr.h
> URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Sema/ParsedAttr.h?rev=339387&r1=339386&r2=339387&view=diff
>
> ==============================================================================
> --- cfe/trunk/include/clang/Sema/ParsedAttr.h (original)
> +++ cfe/trunk/include/clang/Sema/ParsedAttr.h Thu Aug  9 14:13:46 2018
> @@ -118,7 +118,9 @@ class ParsedAttr final
>      : private llvm::TrailingObjects<
>            ParsedAttr, ArgsUnion, detail::AvailabilityData,
>            detail::TypeTagForDatatypeData, ParsedType,
> detail::PropertyData> {
> -  friend class TrailingObjects;
> +  friend class llvm::TrailingObjects<
> +      ParsedAttr, ArgsUnion, detail::AvailabilityData,
> +      detail::TypeTagForDatatypeData, ParsedType, detail::PropertyData>;
>

You need to use "friend TrailingObjects;" here, not "friend class
TrailingObjects;", to avoid breaking MSVC (which doesn't implement
injected-class-names quite according to spec).


>    size_t numTrailingObjects(OverloadToken<ArgsUnion>) const { return
> NumArgs; }
>    size_t numTrailingObjects(OverloadToken<detail::AvailabilityData>)
> const {
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180809/e8829433/attachment.html>


More information about the cfe-commits mailing list