commit f20729914d87ab30e6c96420720d8718affd587f
author George Burgess IV <gbiv@google.com> Tue May 23 15:36:41 2017 -0700
committer George Burgess IV <gbiv@google.com> Tue May 23 15:42:07 2017 -0700
tree d86efb150d9493c97cee50c2c2e9137ec7e4b309
parent 14538fb50832ac20445af5a92003bc250f486c22

optimizing: Fix a potential nullptr dereference

Operator precedence in C++ means the DCHECK was parsed as
DCHECK((allocation != nullptr && allocation->IsNewInstance()) ||
       allocation->IsNewArray())

Caught by the static analyzer:
art/compiler/optimizing/instruction_builder.cc:1004:14: warning: Called
C++ object pointer is null [clang-analyzer-core.CallAndMessage]

Bug: None
Test: Reran the static analyzer. No more nullptr complaints. m builds
the file fine, as well.
Change-Id: Iad94719fc013f1883746128e066452994740f171