lebedev.ri added a comment.

In https://reviews.llvm.org/D37808#869879, @JonasToth wrote:

> In https://reviews.llvm.org/D37808#869823, @lebedev.ri wrote:
>
> > What about GNU extension `case 1 ... 3:` ?
>
>
> Strictly speaking, the coding standard (which should be enforced by the 
> patch) requires strict ISO C++11, therefore this extension is not considered 
> directly.




> Does clang support this extension?

It does:

  test.cpp:3:12: warning: use of GNU case range extension [-Wgnu-case-range]
      case 1 ... 3:
             ^



> When this would be a warning, then that case should be added as well.




https://reviews.llvm.org/D37808



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

Reply via email to