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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
