Annotate possibly unintended fallthroughs in Hexagon MC code, NFC
authorReid Kleckner <rnk@google.com>
Thu, 1 Nov 2018 19:32:04 +0000 (19:32 +0000)
committerReid Kleckner <rnk@google.com>
Thu, 1 Nov 2018 19:32:04 +0000 (19:32 +0000)
commitbebc53f838c38fc2d3cd9e990fa0284497beae61
treee1889e3cdbcca29f633a28803b6ee387985f1c72
parentc8bd6ea35e459169cbd401372e81168ed8482536
Annotate possibly unintended fallthroughs in Hexagon MC code, NFC

Clang's -Wimplicit-fallthrough check fires on these switch cases. GCC
does not warn when a case body that ends in a switch falls through to a
case label of an outer switch.

It's not clear if these fall throughs are truly intended.  The Hexagon
tests pass regardless of whether these case blocks fall through or
break.

For now, I have applied the intended fallthrough annotation macro with a
FIXME comment to unblock enabling the warning. I will send a follow-up
patch that converts them to breaks to the Hexagon maintainers.

llvm-svn: 345878
llvm/lib/Target/Hexagon/MCTargetDesc/HexagonMCDuplexInfo.cpp