Makes sense, LGTM

================
Comment at: test/SemaCXX/switch-implicit-fallthrough.cpp:198-201
@@ -199,6 +197,6 @@
   switch (sizeof(p)) {
     case 9:                    // this test will not work on compilers with 
72-bit long
       n += static_cast<int>(p >> 32);
       [[clang::fallthrough]];  // no warning here
     case 5:                    // it is not intended to work on compilers with 
40-bit long as well
       n += static_cast<int>(p);
----------------
We can drop these comments now.


http://llvm-reviews.chandlerc.com/D374

BRANCH
  svn

ARCANIST PROJECT
  clang
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to