Revert "Revert "Temporarily disable dex cache array emptyness DCHECK""

This fix has been reverted.

Bug: 29083330

This reverts commit 06cd763b7d2a7511ea4a54b52b2931e57f3522d3.

Change-Id: I5382c872caeb7e699de41acaa54ac3ee9f0543e0

diff --git a/runtime/class_linker.cc b/runtime/class_linker.cc
index fe6b14c..e789db6 100644 (file)
--- a/runtime/class_linker.cc
+++ b/runtime/class_linker.cc
@@ -2092,7 +2092,8 @@ mirror::DexCache* ClassLinker::AllocDexCache(Thread* self,
       reinterpret_cast<ArtMethod**>(raw_arrays + layout.MethodsOffset());
   ArtField** fields = (dex_file.NumFieldIds() == 0u) ? nullptr :
       reinterpret_cast<ArtField**>(raw_arrays + layout.FieldsOffset());
-  if (kIsDebugBuild) {
+  // Temporarily disabled since the JIT may hit this case. b/29083330
+  if (kIsDebugBuild && (false)) {
     // Sanity check to make sure all the dex cache arrays are empty. b/28992179
     for (size_t i = 0; i < dex_file.NumStringIds(); ++i) {
       CHECK(strings[i].Read<kWithoutReadBarrier>() == nullptr);