commit d79209ceb8a9d41a4f1c194ed5f9cac111e52835
author Elliott Hughes <enh@google.com> Fri Nov 16 09:29:48 2018 -0800
committer Elliott Hughes <enh@google.com> Mon Nov 26 10:36:59 2018 -0800
tree 2e50a8ccf782366cfce41a9dd801f0d81e8e2cec
parent f3d71d90813be42eb2bc5b85a38cd31dd3c5af21

C++17 compatibility: unreachable `return`.

Clang rejects the `return` as unreachable in C++17 mode, but complains
about falling off the end of a function in C++14 mode. Add UNREACHABLE
to make it happy either way.

Bug: http://b/111067277
Test: builds
Change-Id: I119a806f41e88752976d6fd3638146ec6e08d620

runtime/gc/accounting/mod_union_table_test.cc

diff --git a/runtime/gc/accounting/mod_union_table_test.cc b/runtime/gc/accounting/mod_union_table_test.cc
index b39628b..e66a174 100644
--- a/runtime/gc/accounting/mod_union_table_test.cc
+++ b/runtime/gc/accounting/mod_union_table_test.cc
@@ -162,9 +162,9 @@
     }
     default: {
       UNIMPLEMENTED(FATAL) << "Invalid type " << type;
+      UNREACHABLE();
     }
   }
-  return nullptr;
 }
 
 TEST_F(ModUnionTableTest, TestCardCache) {