majnemer accepted this revision.
majnemer added a comment.
This revision is now accepted and ready to land.

LGTM as-is, clang's behavior would be congruent with GCC's.


================
Comment at: lib/Parse/ParseDeclCXX.cpp:3024-3026
@@ -3013,2 +3023,5 @@
       Diag(FinalLoc, diag::ext_ms_sealed_keyword);
+    else if (Specifier == VirtSpecifiers::VS_GNU_Final) {
+      Diag(FinalLoc, diag::ext_warn_gnu_final);
+    }
 
----------------
Please remove these braces.


Repository:
  rL LLVM

https://reviews.llvm.org/D22919



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to