rsmith added a comment.
In https://reviews.llvm.org/D53985#1284282, @aaron.ballman wrote:
> 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?
>
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
This revision was automatically updated to reflect the committed changes.
Closed by commit rL345870: Use C++11 fallthrough attribute syntax when
available and add a break (authored by rnk, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
rnk added subscribers: aaron.ballman, rsmith.
rnk added a comment.
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
ldionne accepted this revision.
ldionne added a comment.
This revision is now accepted and ready to land.
Would it make sense to add the GNU spelling to the attribute in Clang?
https://reviews.llvm.org/D53985
___
cfe-commits mailing list
rnk created this revision.
rnk added reviewers: EricWF, ldionne.
Herald added subscribers: erik.pilkington, christof.
This silences the two -Wimplicit-fallthrough warnings clang finds in
ItaniumDemangle.h in libc++abi.
Clang does not have a GNU attribute spelling for this attribute, so this
is