[PATCH] D69935: [DeclCXX] Remove unknown external linkage specifications

Tue Nov 12 03:53:03 PST 2019

uabelho added a comment.

In D69935#1739235 <https://reviews.llvm.org/D69935#1739235>, @dblaikie wrote:

> In D69935#1738273 <https://reviews.llvm.org/D69935#1738273>, @uabelho wrote:
>
> > Don't you need to also remove
> >
> >   case LinkageSpecDecl::lang_cxx_11:
> >   case LinkageSpecDecl::lang_cxx_14:
> >   
> >
> > from VisitLinkageSpecDecl in clang-tools-extra/modularize/Modularize.cpp? (added in r372714 / e07376a320d to silence a clang warning).
> >  I can't see how that code would compile otherwise?
>
>
> +1 to this

Did you see the above?

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

https://reviews.llvm.org/D69935