diff options
| author | Ian Rogers <irogers@google.com> | 2014-10-09 21:56:44 -0700 |
|---|---|---|
| committer | Ian Rogers <irogers@google.com> | 2014-10-09 22:22:46 -0700 |
| commit | fc787ecd91127b2c8458afd94e5148e2ae51a1f5 (patch) | |
| tree | ef48c0f511ee9bf4ed85607cc4d530bace7e6cae /runtime/hprof | |
| parent | 8fa8c904f7c783204a1dc9438429391d256658da (diff) | |
| download | art-fc787ecd91127b2c8458afd94e5148e2ae51a1f5.zip art-fc787ecd91127b2c8458afd94e5148e2ae51a1f5.tar.gz art-fc787ecd91127b2c8458afd94e5148e2ae51a1f5.tar.bz2 | |
Enable -Wimplicit-fallthrough.
Falling through switch cases on a clang build must now annotate the fallthrough
with the FALLTHROUGH_INTENDED macro.
Bug: 17731372
Change-Id: I836451cd5f96b01d1ababdbf9eef677fe8fa8324
Diffstat (limited to 'runtime/hprof')
| -rw-r--r-- | runtime/hprof/hprof.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/runtime/hprof/hprof.cc b/runtime/hprof/hprof.cc index fd67197..a2d37b3 100644 --- a/runtime/hprof/hprof.cc +++ b/runtime/hprof/hprof.cc @@ -719,9 +719,9 @@ static HprofBasicType SignatureToBasicTypeAndSize(const char* sig, size_t* sizeO case 'D': ret = hprof_basic_double; size = 8; break; case 'B': ret = hprof_basic_byte; size = 1; break; case 'S': ret = hprof_basic_short; size = 2; break; - default: CHECK(false); case 'I': ret = hprof_basic_int; size = 4; break; case 'J': ret = hprof_basic_long; size = 8; break; + default: LOG(FATAL) << "UNREACHABLE"; UNREACHABLE(); } if (sizeOut != NULL) { @@ -742,9 +742,9 @@ static HprofBasicType PrimitiveToBasicTypeAndSize(Primitive::Type prim, size_t* case Primitive::kPrimDouble: ret = hprof_basic_double; size = 8; break; case Primitive::kPrimByte: ret = hprof_basic_byte; size = 1; break; case Primitive::kPrimShort: ret = hprof_basic_short; size = 2; break; - default: CHECK(false); case Primitive::kPrimInt: ret = hprof_basic_int; size = 4; break; case Primitive::kPrimLong: ret = hprof_basic_long; size = 8; break; + default: LOG(FATAL) << "UNREACHABLE"; UNREACHABLE(); } if (sizeOut != NULL) { |