[PATCH] D53985: Use C++11 fallthrough attribute syntax when available and add a break

Aaron Ballman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 1 11:34:32 PDT 2018


aaron.ballman added a comment.

In https://reviews.llvm.org/D53985#1284246, @rnk wrote:

> In https://reviews.llvm.org/D53985#1284196, @ldionne wrote:
>
> > Would it make sense to add the GNU spelling to the attribute in Clang?
>
>
> I'll redirect that question to @rsmith and @aaron.ballman. I have a vague recollection that it was considered and rejected, but that decision may have predated GCC 7.


I'm not opposed to it given that it's an attribute supported by GCC.


Repository:
  rL LLVM

https://reviews.llvm.org/D53985





More information about the llvm-commits mailing list