hans accepted this revision. hans added a comment. This revision is now accepted and ready to land.
lgtm since it matches xcode clang For the commit message, perhaps the "(Xcode's clang already behaves like this," part should be moved further down, probably to just after "This patch makes it so that", since the "already behaves like this" should refer to the new behaviour. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D132258/new/ https://reviews.llvm.org/D132258 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits